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

[a11y] Improve the accesbility of the AdvancedDrawer widget #62

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

luke-genvis
Copy link

@luke-genvis luke-genvis commented Mar 10, 2025

Description of the Issue

Currently, the AdvancedDrawer widget has several accessibility issues that are difficult to resolve outside the package:

Reference: Flutter Accessibility Documentation

Changes

  • Added drawerCloseSemanticLabel field to provide a semantic label for the close button
  • Implemented ExcludeFocus and ExcludeSemantics widgets in appropriate places
  • Made minor optimizations, such as using Visibility widget where possible

Visual Demonstration

Screen Reader Navigation

Before After
screen-20250310-224548.mp4
screen-20250310-224308.mp4

External Bluetooth Keyboard Navigation

Note

The focus_debugger package was used to visualize the focused element.

It should be noted debugger has a issue where the box occupies the entire screen when focus is lost. In the video, this just indicates that the focus node element no longer exists in the focus tree (Due to the addition of ExcludeFocus)

Before After
screen-20250310-224645.mp4
screen-20250310-224745.mp4

@luke-genvis luke-genvis changed the title Improve the accesbility of the AdvancedDrawer widget [a11y] Improve the accesbility of the AdvancedDrawer widget Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant