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.

Sign up for GitHub

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

Jump to bottom

[docs][base][Menu] Keyboard navigation doesn't work in Hooks demo #38389

Closed
2 tasks done
homerchen19 opened this issue Aug 9, 2023 · 1 comment · Fixed by #38479
Closed
2 tasks done

[docs][base][Menu] Keyboard navigation doesn't work in Hooks demo #38389

homerchen19 opened this issue Aug 9, 2023 · 1 comment · Fixed by #38479
Assignees
@michaldudak
Labels
bug 🐛 Something doesn't work docs Improvements or additions to the documentation package: base-ui Specific to @mui/base

Comments

@homerchen19
Copy link
Contributor

homerchen19 commented Aug 9, 2023 •
edited
Loading

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Steps to reproduce 🕹

Link to live example:

https://mui.com/base-ui/react-menu/#hooks

Steps:

  1. Focus on the menu button ("Theme") and press "Enter"
  2. Press "Arrow up" or "Arrow down"
  3. Menu items are not focused

Screenshot 2023-08-09 at 14 53 39

Current behavior 😯

Only the Hooks demo has this issue. Other demos work well.

Expected behavior 🤔

Menu items should get focused when I press "Arrow up" or "Arrow down"

Context 🔦

No response

Your environment 🌎

"@mui/base": "5.0.0-beta.10"
@homerchen19 homerchen19 added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Aug 9, 2023
@zannager zannager added docs Improvements or additions to the documentation package: base-ui Specific to @mui/base labels Aug 9, 2023
@michaldudak
Copy link
Member

Thanks for reporting this. It seems that after opening the menu, focus isn't moved to the first item. Would you like to take a look at the code and investigate it?

@michaldudak michaldudak added bug 🐛 Something doesn't work and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Aug 11, 2023
@homerchen19 homerchen19 mentioned this issue Aug 15, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaldudak michaldudak

Labels
bug 🐛 Something doesn't work docs Improvements or additions to the documentation package: base-ui Specific to @mui/base
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[docs][base-ui] Fix Menu Hooks demo homerchen19/material-ui
3 participants
@michaldudak @homerchen19 @zannager