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

[Menu] openOnHover issues #1180

Open
atomiks opened this issue Dec 19, 2024 · 0 comments · May be fixed by #1191
Open

[Menu] openOnHover issues #1180

atomiks opened this issue Dec 19, 2024 · 0 comments · May be fixed by #1191
Labels
bug 🐛 Something doesn't work component: menu This is the name of the generic UI component, not the React module!

Comments

@atomiks
Copy link
Contributor

atomiks commented Dec 19, 2024

For Menu openOnHover demo

1. First item incorrectly highlighted when mixing pointer/keyboard use

  • Focus trigger using keyboard
  • Press esc
  • Hover the trigger with pointer
  • First item highlighted incorrectly (nothing should be highlighted when using pointer)

2. Returns focus and scrolls to element out of view

  • Focus 'Group Labels' demo trigger (View) at the bottom with keyboard
  • Scroll up and hover the openOnHover demo
  • Mouseout of the trigger
  • Browser focuses and scrolls down to the 'Group Labels' trigger (focus management should be disabled like Popover when using openOnHover)

3. Stick behavior

  • Hover openOnHover demo Menu
  • Click
  • Mouseout of trigger
  • Stays stuck but should not (it also shouldn't stick if you moused into the popup, but didn't click anything)

Side note: I think Popover's implementation can also be improved in that clicking within the patient threshold makes it stick, but after 500ms (the current value) it closes, but they should at least match each other.

@atomiks atomiks added component: menu This is the name of the generic UI component, not the React module! bug 🐛 Something doesn't work labels Dec 19, 2024
@atomiks atomiks linked a pull request Dec 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: menu This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant