You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I like the idea of highlighting shortcuts, even though I'm pressing alt all the time to use accessibility tools. That may be why I realized that there seem to be a number of things missing. It seems odd that e.g. “Shortcuts” has a shortcut, but the buttons right next to that do not.
This also goes for other places in the editor like e.g. the settings (gear) icons.
Fair. I'm happy to add more shortcuts in the future. I'm not sure all places you mention really deserve their own shortcut, but that's a discussion we can have after this PR is merged.
The problem is mostly that you introduced the shortcut overlay, which I think looks very odd in its current form. At least on the main menu.
I'm not sure whether this issue is supposed to be mainly about the missing/additional shortcuts, or whether @lkiesow's last comment in particular wants to express some more concerns. Can you elaborate?
The text was updated successfully, but these errors were encountered:
To give you a specific example, whenever you press <alt> (I do that all the time to access some accessibility options) you now get an overlay showing you potential shortcuts. That's new. Before you just had some shortcuts for functionality that people deemed important. This leads to the user interface looking like this (if you look, for example, to the main menu):
So, I get an overlay over one of four icons in the main menu. That looks pretty weird.
What's the solution to this? You could remove the overlay again. Or you could add shortcuts to all menu items. It's just that in it's current form, it's not great.
@lkiesow wrote in #1069 (comment):
@LukasKalbertodt responded in #1069 (comment):
@lkiesow added in #1069 (comment):
I'm not sure whether this issue is supposed to be mainly about the missing/additional shortcuts, or whether @lkiesow's last comment in particular wants to express some more concerns. Can you elaborate?
The text was updated successfully, but these errors were encountered: