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

[Header] Hidden mobile content focusable #769

Closed
svinkle opened this issue Oct 13, 2021 · 0 comments · Fixed by #1493
Closed

[Header] Hidden mobile content focusable #769

svinkle opened this issue Oct 13, 2021 · 0 comments · Fixed by #1493
Assignees
Labels
Category: Accessibility Bugs found while using assistive technology Critical Path: 1 Affects all merchants/buyers Effort: High Severity: 3 Normal/Low Severity

Comments

@svinkle
Copy link
Member

svinkle commented Oct 13, 2021

Issue summary

When a mobile submenu is active, visually hidden links below the currently visible menu are discoverable when using swipe navigation or the virtual cursor. This may result in a confusing user experience for screen reader users who should only be able to access the current set of visible links.

Screenshots Dawn mobile menu. VoiceOver output shows a focused link which is under the currently visible submenu.

Steps to reproduce

  1. Go to a Dawn theme store with any supported mobile browser
  2. Open the main navigation, then a sub-navigation section
  3. With an active screen reader, navigate through the content via swipe gestures

Behavior

Expected

  • Only visible content to be discoverable.

Actual

  • Links "under" the current set of visible links are discoverable using swipe gestures.

Recommendation

Set any underlying content as visibility: hidden in order to prevent discoverability.

Specifications

  • Browser: Any
  • Dawn version: 2.2.0
  • Component: Collection page
  • WCAG Principal: Operable
  • WCAG SC: 2.4.3: Focus Order
  • Severity: Medium
  • Effort: High
@svinkle svinkle added Severity: 3 Normal/Low Severity Category: Accessibility Bugs found while using assistive technology Effort: High labels Oct 13, 2021
@svinkle svinkle added the Critical Path: 1 Affects all merchants/buyers label Nov 26, 2021
@kmeleta kmeleta self-assigned this Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Accessibility Bugs found while using assistive technology Critical Path: 1 Affects all merchants/buyers Effort: High Severity: 3 Normal/Low Severity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants