nurpax / petmate

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

Tooltip text for draw modes #76

Closed Esshahn closed 6 years ago

Esshahn commented 6 years ago

Take these as a suggestion or to get refinement started:

screenshot 2018-08-12 um 10 55 12

nurpax commented 6 years ago

IMAGE ALT TEXT HERE

(click for a video)

Abbreviated 'character' to just 'char'. Just to keep the tooltips bit smaller. Tooltips might benefit from a bit of styling, the black+white scheme looks too constrasty compared to the rest of the app.

nurpax commented 6 years ago

@Esshahn I wonder if the brush icon (or brush tool name, even) is a bit confusing. At least the icon might be better served with the usual crop and/or select area icon (e.g. https://fontawesome.com/icons/crop?style=solid)

Esshahn commented 6 years ago

Agreed. Not so sure about the crop icon (as we're not cropping). Maybe something more like these? clone: https://fontawesome.com/icons/clone?style=regular stamp: https://fontawesome.com/icons/stamp?style=solid

nurpax commented 6 years ago

I like the stamp icon. I'll switch it to that. Should we rename the tool to stamp or keep it as brush?

nurpax commented 6 years ago
screen shot 2018-08-14 at 12 49 30 am
Esshahn commented 6 years ago

Stamp, or stencil maybe. Would both work I guess. Pick what you prefer.

nurpax commented 6 years ago

I tried changing it to stamp.. but when I was updating the docs, somehow brush just felt better than "stamp" or "stencil". Maybe I'll just keep it as it is.