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

Sidebar menu opened by tab + enter does not receive focus #7118

Open
2 tasks
andrehgdias opened this issue Jun 21, 2024 · 0 comments · May be fixed by #7123
Open
2 tasks

Sidebar menu opened by tab + enter does not receive focus #7118

andrehgdias opened this issue Jun 21, 2024 · 0 comments · May be fixed by #7123
Assignees
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done

Comments

@andrehgdias
Copy link
Contributor

andrehgdias commented Jun 21, 2024

Describe the bug
In a single or two-column layout, if you try opening and navigating through your sidebar menu (index, drafts, etc) using Tab key you can't focus on the menu itself so the focus stays in the email list even though the sidebar is open and blocking the view.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the mail list (or any other page) while in a single or two-column layout
  2. Press Tab until you highlight the hamburger menu icon and press Enter
  3. Try tabbing two navigate between the menu options.

Expected behavior
Navigate through the menu options

Test Notes

  • On single and two-columns layout, when opening sidebar through hamburg menu, we should be able to tab inside it.
  • Tabbing anywhere else on the application should not be allowed while the sidebar is open.
@andrehgdias andrehgdias added the bug broken functionality, usability problems, unexpected errors label Jun 21, 2024
@charlag charlag added the state:done meets our definition of done label Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants