search

nurpax / petmate

PETSCII editor with Electron/React/Redux
MIT License
182 stars 14 forks source link

better icon for "clear canvas" #57

Closed nurpax closed 6 years ago

nurpax commented 6 years ago

preferably choose one from fontawesome.com

Viza74 commented 6 years ago

"file" - rotated 90? "sticky note"? some from the different variantions of "times" X? maaaaaybe "tint-slash"?

Or maybe just delete that tool? It's placement is way to prominent for it's function anyway - you basically never need it. Also the "clear canvas" action concerns the whole screen, so it could be moved to the right click menu in the screens toolbar too.

nurpax commented 6 years ago
screen shot 2018-07-31 at 12 09 49 am

I went with the cute broomstick icon for now :)

Agreed that it may be a bit redundant now.. The make a new empty screen, it's enough to click on the plus icon on the top bar.

nurpax commented 6 years ago

Off-topic: the PM logo is WIP from Cal. Not sure if this is the one we go with.

nurpax commented 6 years ago

Also moved undo/redo to top. I think they're better there than between clear and the rest of the drawing operations.

As for clear.. A rectangle clear would be useful. Maybe it'd make more sense to add clear rect, colorize rect tools and get rid of clear canvas.