artursapek / mondrian

Web-based vector graphics editor

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Show tooltips when hovering the tool icons

waldyrious opened this issue · comments

👍, this would be a great addition.

Aesthetically,

  • they should match the UI of course
  • it should be something like "name (hotkey)" eg "Pen (P)"
  • there should be a delay before they fade in

More importantly, they should be abstracted so that they're easy to change in a tool's config. Eg a tool should have a hotkey key, and a name key that populate the tooltip. So for tools.pen, that would look like:

  name: "Pen"
  hotkey: "p"

Currently tools don't define their own hotkeys - they are hard-coded here. Ideally it would work how menu items work, where they define their own hotkey and the class takes care of listening to it. I never got around to refactoring the tools to work the same way, so this is a good opportunity to do that.

As discussed via Email: I'm taking this up.

Yes, I had found the hotkeys class, and had wondered if class-defined hotkeys wouldn't be better.

Eventually, it would probably be best to have rebindable keys. Perhaps we can have one interface where the tools initially bind themselves to a key, and then settings are loaded, rebinding them as the user defines.

@artursapek So as I understand it, these are duplicates and should be removed?

Yes, user-customized bindings shouldn't be hard to implement. Not really a priority for now, though. The hotkeys are pretty standard and memorable (many of them even match Illustrator's).

And I do think those are duplicates. Just try removing them and see if those functions break, the app will give you a more definite answer than me :)