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

Report aria-haspopup dialog/grid/list/tree in Firefox and Chrome. #14709

Merged
merged 2 commits into from
Mar 9, 2023

Conversation

jcsteh
Copy link
Contributor

@jcsteh jcsteh commented Mar 8, 2023

Link to issue number:

Fixes #8235.

Summary of the issue:

NVDA reports "sub menu" for all values of aria-haspopup, which is misleading at best.

Description of user facing changes

In Mozilla Firefox and Google Chrome, NVDA now reports when a control opens a dialog, grid, list or tree if the author has specified this using aria-haspopup.

Description of development approach

  1. Created new states for HASPOPUP_DIALOG, GRID, LIST and TREE. The general HASPOPUP state is still used for menus.
  2. Mapped the ARIA string values to states.
  3. Modified both the IA2Web NVDAObject and the Gecko_ia2 vbuf to expose these new states based on the haspopup object attribute.

Testing strategy:

Tested with the test case in #8235. The correct message - "opens dialog", etc. - is reported when tabbing in focus mode, tabbing in browse mode, moving by line in browse mode and quick navigating in browse mode.

Known issues with pull request:

None.

Change log entries:

New features
In Mozilla Firefox and Google Chrome, NVDA now reports when a control opens a dialog, grid, list or tree if the author has specified this using aria-haspopup.

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@jcsteh jcsteh requested a review from a team as a code owner March 8, 2023 14:39
@jcsteh jcsteh requested a review from seanbudd March 8, 2023 14:39
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.

NVDA does not recognize the ARIA 1.1 values of aria-haspopup
3 participants