Open Bug 1939126 Opened 2 months ago Updated 13 days ago

Firefox Relay text in alert is not announced by NVDA

Categories

(Toolkit :: Password Manager, defect)

Firefox 135
x86_64
Windows 10
defect

Tracking

()

Accessibility Severity s3
Tracking Status
firefox135 --- affected

People

(Reporter: nstroud, Unassigned, NeedInfo)

References

Details

(Keywords: access, Whiteboard: [fxcm-relay-all])

Attachments

(1 file)

Prerequisites:

Found in Nightly 135.0a1 (2024-12-25)(64-bit)
Follow 'How to Test' steps in this Google Doc

STR:

  1. Turn on NVDA
  2. Open Firefox Nightly
  3. Open Registration Forms page
  4. Using the TAB key, enter the email text input box under 'Registration form with username & password'. The autocomplete box will appear
  5. Press the Down arrow key and press ENTER. The Firefox Relay alert dialog will appear. Observe NVDA speech output

Expected / Actual:

At this time, NVDA only says 'alert'. aria-labelledby should be added to the Firefox Relay doorhanger and id should be added to the Firefox Relay text. aria-labelledby should point to this id

There's no priority nor severity set here. What's the process for assigning priority and severity to a11y bugs?

Flags: needinfo?(nstroud)

Hi :groovecoder! We've set the Accessibility Severity above. We only set the overall severity for bugs in the Disability Access API (platform) and Accessibility Inspector in DevTools. Hope this helps!

Accessibility/Triage - Mozilla Wiki

Flags: needinfo?(nstroud) → needinfo?(lcrouch)

:groovecoder do you know why this isn't propagated to Jira?

Whiteboard: [fxcm-relay-all]

(In response to Santiago's comment on FXCM-1563- I'm not able to respond in JIRA)

Hi Santiago! Please see the responses below!

Do you have a suggest for what it should say instead of “Alert”? Do know what other messages using the doorhanger say?
+
I noticed that it doesn’t actually read the title - isn’t it an expectation that it should?
-- I'm going to answer these 2 together :) Nothing needs to be changed with NVDA announcing 'alert' as this is the role. Plus the announcement lets screen reader users know that important information has appeared on page. With that said, you're correct with the 2nd question! We have the role (alert) and need a name which is the title of the alert ("Prevent spam with a free email mask"). This is where we want to use aria-labelledby. The container will need the aria-labelledby attribute that points to an id attribute attached to the title. Here is the aria-labelledby page on MDN Web Docs for more information on how to implement this

Is it a problem that both the “Next” and “Not now” buttons have the “n” underlined? I noticed one can toggle between them by pressing the alt+n but is the best practice to not have the same letters? If so, do you know what’s likely better?
-- I'm checking into this one!

Repeat email mask + email mask link at the end:
-- Which modal is this one? I couldn't find it but upon testing 'Prevent spam with a free email mask', a similar thing happens

Flags: needinfo?(sandrigo)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: