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

[bug]: Focus handling mobile navigation #4546

Closed
gfellerph opened this issue Jan 29, 2025 · 0 comments · Fixed by #4625
Closed

[bug]: Focus handling mobile navigation #4546

gfellerph opened this issue Jan 29, 2025 · 0 comments · Fixed by #4625
Assignees
@schaertim
Labels
🐞 bug Something isn't working 📦 components Related to the @swisspost/design-system-components package Header Everything related to our composible header components
Milestone
PPNL Support

Comments

@gfellerph
Copy link
Member

Describe the bug
When opening the second level navigation, the focus should jump to the first focusable element on the second level. Currently, the focus stays on the first level trigger and becomes invisible as soon as the second level slides in.

When the second level gets closed, the focus should jump back to the first level trigger that opened the second level navigation. Currently, the focus stays on the hidden second level navigation.

To Reproduce
Steps to reproduce the behavior:

  1. Open main navigation on mobile
  2. Click tab until focus is on first level entry with a megadropdown
  3. Click enter
  4. The focus is now hidden
@gfellerph gfellerph added 🐞 bug Something isn't working 📦 components Related to the @swisspost/design-system-components package labels Jan 29, 2025
@gfellerph gfellerph added this to the PPNL Support milestone Jan 29, 2025
@alionazherdetska alionazherdetska added the Header Everything related to our composible header components label Feb 4, 2025
@gfellerph gfellerph moved this from 👀 Triage to 💻 Ready for development in Design System Production Board Feb 5, 2025
@schaertim schaertim self-assigned this Feb 6, 2025
@schaertim schaertim linked a pull request Feb 6, 2025 that will close this issue
chore(header): impoved focus behaviour of megadropdown #4625
Merged
@schaertim schaertim moved this from 💻 Ready for development to 🕹️ Coding in progress in Design System Production Board Feb 6, 2025
@schaertim schaertim moved this from 🕹️ Coding in progress to 🤬 Dev in Code Review in Design System Production Board Feb 6, 2025
@github-project-automation github-project-automation bot moved this from 🤬 Dev in Code Review to 🚀 Done in Design System Production Board Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schaertim schaertim

Labels
🐞 bug Something isn't working 📦 components Related to the @swisspost/design-system-components package Header Everything related to our composible header components
Projects
Design System Production Board
Status: 🚀 Done
Milestone
PPNL Support
Development

Successfully merging a pull request may close this issue.

chore(header): impoved focus behaviour of megadropdown swisspost/design-system
3 participants
@gfellerph @schaertim @alionazherdetska