Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explore more obvious section headers in quick pick #149057

Open
daviddossett opened this issue May 9, 2022 · 23 comments
Open

Explore more obvious section headers in quick pick #149057

daviddossett opened this issue May 9, 2022 · 23 comments
Assignees
Labels
under-discussion Issue is under discussion for relevance, priority, approach ux User experience issues

Comments

@daviddossett
Copy link
Contributor

The right-aligned label text has always been slightly confusing to me as a pattern to separate sections in quick pick lists. I think it's worth exploring how we might make these more obvious while still using vertical space efficiently.

Current

CleanShot 2022-05-08 at 21 29 37@2x

Examples

GitHub

CleanShot 2022-05-08 at 21 30 48@2x

Raycast

CleanShot 2022-05-08 at 21 31 09@2x

cc @lychung7 @misolori

@daviddossett daviddossett added ux User experience issues under-discussion Issue is under discussion for relevance, priority, approach labels May 9, 2022
@daviddossett daviddossett self-assigned this May 9, 2022
@jaeyonglee8687
Copy link

Sharing another example

Runner Pro
https://docs.sketchrunner.com/search-commands/run
image

@daviddossett
Copy link
Contributor Author

A couple of possible solutions within the current quick pick design:

CleanShot 2022-06-10 at 10 52 15@2x

CleanShot 2022-06-10 at 10 52 20@2x

@daviddossett
Copy link
Contributor Author

This could also be extended into an idea I had for a "comfortable" density mode for the quick pick/command palette.

CleanShot 2022-05-17 at 20 50

@daviddossett
Copy link
Contributor Author

daviddossett commented Aug 8, 2022

Extending these explorations to include ideas for actions in the section header rows. Some early ideas:

With Inline Icon Actions

with-section-headers

With Icon Action Labels

We could also use labels here since the inline icons alone would be a downgrade in terms of understanding what the command is—today we use the plain text as a quick pick entry so this could retain some of that clarity.

with-action-labels

cc @misolori @TylerLeonhardt

@TylerLeonhardt
Copy link
Member

I like that the separator buttons could have text to them though I'm sure the first feature request once that lands is to allow buttons in the normal quick pick items to have text :) not a bad idea but I am concerned about space in the quickpick...

@miguelsolorio
Copy link
Contributor

I think starting with icons (no labels) is a good start to match most of our inline actions. I also wonder if we can have the section header actions/toolbar appear when you are on a section, just like views?

CleanShot 2022-08-10 at 10 46 28@2x

@daviddossett
Copy link
Contributor Author

Agreed.

I also wonder if we can have the section header actions/toolbar appear when you are on a section, just like views?

By that do you mean like if you're navigating via keyboard we should up the parent actions? 👍 if so!

@miguelsolorio
Copy link
Contributor

Right now we show the toolbar whenever you are hovering in a view and then it hides when you are off, as opposed to only showing it when you are hovering over the section title.

@daviddossett
Copy link
Contributor Author

Oh ok even better. Great idea.

@jaeyonglee8687
Copy link

Love the iterations. The blue color title seems to have a function or embedded link it it at the first glance. It worked when it was on the same line with a quickPick item, but it becomes a little confusing with the separation between section title and pickPick items.

@daviddossett
Copy link
Contributor Author

daviddossett commented Aug 11, 2022

The blue color title seems to have a function or embedded link it it at the first glance

That's a fair point. Darker or lighter greys can in some ways make it harder to distinguish between sections. Maybe some spacing above each section and retaining the border would help here:

CleanShot 2022-08-11 at 11 10 44@2x

@daviddossett
Copy link
Contributor Author

Incoroporated a few bits of feedback and fixes:

  • Added dedicated divder rows to help distinguish between sections
  • Added @misolori's suggestion to show the actions on hover/focus of children

CleanShot 2022-08-15 at 09 32 47

@TylerLeonhardt
Copy link
Member

TylerLeonhardt commented Aug 15, 2022

@daviddossett two questions:

  • Do you think we should keep the existing look if no separators have buttons?
  • If so... If only a single separator has buttons, should they the UI of separators be aligned to all take up an entire row even if they don't have buttons?

@daviddossett
Copy link
Contributor Author

Do you think we should keep the existing look if no separators have buttons?

My gut feeling is that it should be a global change. There are a bunch of differences in font, spacing, etc. that makes me think it would be confusing for users to see different formats across different lists. Here's how it would look on a regular grouped quick pick list:

CleanShot 2022-08-15 at 10 22 14@2x

Some inspiration from Safari for those worried about keyboard interactions. Feels super natural to arrow up/down across sections:

CleanShot 2022-08-15 at 10 54 03

@eamodio
Copy link
Contributor

eamodio commented Aug 16, 2022

I like the latest iteration. Although I think the section headers are too bold -- the blue stands out and draws the eye much more than the actual choices. I would prefer a style closer to the safari example where the headers are more muted, so the focus stays on the choices themselves.

@daviddossett
Copy link
Contributor Author

daviddossett commented Aug 18, 2022

I like the grey headers too, at least in this particular default theme. I've landed on that for now. We're also looking at how to better integrate section-specific actions. Today they're often placed at the bottom of a long quick pick list and can be easily missed.

Header w/ Toolbar

Shows up on hover anywhere in that section. Advantages here are the similarities to a view toolbar and reduced noise in the list. Possible makes it difficult to have a natural-feeling keyboard navigation. We also lose the descriptive list item labels + search capability we have today.

CleanShot 2022-08-18 at 14 38 49

Header with Pinned Action List Items

We could have section-specific actions rendered at the top of the relevant section as normal list items. This could be achieved today but it would obviously need to be done manually. Maybe there's a way to pin items per section?

CleanShot 2022-08-18 at 14 43 33@2x

@usernamehw
Copy link
Contributor

usernamehw commented Oct 9, 2022

Any chance of keeping headers even after typing? For example, Theme Picker with empty input shows:

──────────────────────────────────────────────────────────── light themes
...
GitHub Light
GitHub Light Colorblind
Light (Visual Studio)
Light+ (default light)
...
──────────────────────────────────────────────────────────── dark themes
...
Dark (Visual Studio)
Dark+ (default dark)
GitHub Dark
GitHub Dark Colorblind
...

After filtering they are no longer grouped (input content - github):

GitHub Dark
GitHub Light
GitHub Dark Colorblind
GitHub Light Colorblind
...

Ideally, groups should persist (input content - github):

──────────────────────────────────────────────────────────── light themes
GitHub Light
GitHub Light Colorblind
──────────────────────────────────────────────────────────── dark themes
GitHub Dark
GitHub Dark Colorblind

@daviddossett
Copy link
Contributor Author

@usernamehw I think that's a reasonable proposal but I'd need to see how to approach quick picks that show something different after typing. There are some examples of presenting other content instead of the filtered list.

@TylerLeonhardt
Copy link
Member

@usernamehw that sounds like a bug to me because I get that rough experience:
image

Can you open a new issue and we can get that looked at?

@daviddossett
Copy link
Contributor Author

It does look like there is some strangeness going on here.

CleanShot 2022-10-10 at 09 54 22@2x

@TylerLeonhardt
Copy link
Member

@aeschli is this feedback for you? ^

@aeschli
Copy link
Contributor

aeschli commented Oct 11, 2022

Agree that this looks strange. Filtering & sorting doesn't go with the separator header.
It's probably a bug with quick fix dialog to still show the separator labels when filtering (and sorting)
Only when sortByLabel: false is set the separators can be shown while filtering.
That what is done in the remote indicator menu:
image

@usernamehw
Copy link
Contributor

Is sortByLabel available to extension api?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
under-discussion Issue is under discussion for relevance, priority, approach ux User experience issues
Projects
Status: 🏗 In Design
Development

No branches or pull requests

7 participants