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

6.1 beta Search and sidebar behavior #13000

Closed
domyen opened this issue Nov 3, 2020 · 2 comments
Closed

6.1 beta Search and sidebar behavior #13000

domyen opened this issue Nov 3, 2020 · 2 comments
Assignees
Milestone

Comments

@domyen
Copy link
Member

domyen commented Nov 3, 2020

RFCs for edge cases

  1. Update empty state for search: show a “no results” message. Currently we show no results and no messages which makes the UI appear broken.
    image

  2. No recently opened: When there are no recently opened items continue showing the treeview. Currently we show nothing so it looks like the treeview completely disappeared. You can reproduce this state by clearing recently opened.

image

  1. When you click on the expand all button, keyboard navigation no longer behaves as I'd expect. Every time I press enter the focus returns to the expand all button no matter what I focus on using my keyboard. Video here

  2. Keyboard navigation skips items when a folder is expanded and collapsed. Video here. When expanding and collapsing using the right and left arrow keys I expect the focus to start at the very beginning of the list not in the middle.

  3. Keyboard navigation focus doesn't transfer to the "folder" items. Video here

@ghengeveld
Copy link
Member

  1. Update empty state for search: show a “no results” message. Currently we show no results and no messages which makes the UI appear broken.

I added a "No components found" message:

Screenshot 2020-11-05 at 21 12 17

  1. No recently opened: When there are no recently opened items continue showing the treeview. Currently we show nothing so it looks like the treeview completely disappeared. You can reproduce this state by clearing recently opened.

Instead of showing the treeview, I made sure that the history will always show at least one item. After all, you will always have a component selected, so technically that one should always be at the top of the list.

  1. When you click on the expand all button, keyboard navigation no longer behaves as I'd expect. Every time I press enter the focus returns to the expand all button no matter what I focus on using my keyboard. Video here

Fixed this by removing focus from the expand-all button when using the arrow keys to navigate the tree.

  1. Keyboard navigation skips items when a folder is expanded and collapsed. Video here. When expanding and collapsing using the right and left arrow keys I expect the focus to start at the very beginning of the list not in the middle.

This was a bug and is fixed now.

  1. Keyboard navigation focus doesn't transfer to the "folder" items. Video here

Fixed. Now when you click a node to expand/collapse it, it will get highlighted as well.

@shilman
Copy link
Member

shilman commented Nov 6, 2020

ZOMG!! I just released https://github.com/storybookjs/storybook/releases/tag/v6.1.0-beta.4 containing PR #13027 that references this issue. Upgrade today to the @next NPM tag to try it out!

npx sb upgrade --prerelease

Closing this issue. Please re-open if you think there's still more to do.

@shilman shilman closed this as completed Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants