-
Notifications
You must be signed in to change notification settings - Fork 251
Popup menu buttons UI/UX #402
Comments
This kind of design seems great, it would help to solve many of the issues that arise when trying to add new features. It would also help with #355, because making the header stay above input and parse results on the search page feels wrong. I'm really happy that there's someone who can sketch these kinds of things, because my own experience is limited to poking CSS until it works. |
I'm not sure if this is a separate issue, but I think it could be a problem that scrolling doesn't change the current entry but alt+up/down/scroll does. Many users could be just scrolling through the results until they find the one they wish to make a note of, and click the button that's at the top of the entry. While changing the current entry to the one currently displayed could be made automatic, it's still ambiguous when that change actually happens: when you're half way to the second entry, which one are you viewing? |
Yeah, I've thought about this a few times too, part of the issue stemming from the indicator being offscreen. A common indicator of the "current selection" is a small, colored sidebar: This would be a separate issue from the menu buttons UI task, since there's not much overlap beyond just considering that the designs work well together. |
That design looks absolutely sexy 👍 |
Here is a quick mock-up of a potential design, with some icons as placeholders. This is a fairly minimal design that doesn't feel like it would significantly change users' workflows.
For comparison, if the buttons were placed on the top or left sides, the cursor would need to be used in a different location than the current design, which could impede muscle memory patterns, reduce available layout space, or just generally an undesirable change.
High-level overview of tasks involved in implementing something like this:
This is not necessarily a final design; this issue is intended to discuss needs and evolve the design as necessary.
Related: #292
The text was updated successfully, but these errors were encountered: