Closed Bug 1583428 Opened 4 months ago Closed 4 months ago

Fix image a11y labels in about:logins

Categories

(Firefox :: about:logins, defect, P2)

70 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 71
Tracking Status
firefox70 + verified
firefox71 --- verified

People

(Reporter: Jamie, Assigned: jaws)

Details

(Keywords: access, Whiteboard: [passwords:management] [skyline][access-p2])

Attachments

(1 file)

I'd recommend two tweaks to image labels for accessibility in about:logins:

  1. Favicons currently get an alt attribute with a string like "Favicon for login.yahoo.com". This ends up being extraneous verbosity for screen reader users, since the domain is already provided as text separately. I would instead set this to alt="".
  2. Breach icons get an alt attribute of "Warning icon". That is the label that ends up getting used when the a11y engine builds the label for the list item. However, the more interesting part is the title of "Breached website", which currently does not get included in the label for the list item (because it is shadowed by alt). I would suggest removing the alt attribute and letting the title provide the label.

Please let me know if you'd prefer these as separate bugs an dI'll split one of them out.

Summary: Image a11y labels in about:logins → Fix image a11y labels in about:logins

Keeping as one bug is fine.

Priority: -- → P2
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f8cec7bebba9
Remove the alt attribute for the favicon since it is not helpful and remove the alt attribute on the breached website icon so the title of 'Breached Website' is spoken. r=fluent-reviewers,MattN,flod
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

I have verified this issue on the latest Nightly (Build ID: 20191003214801) on Windows 7 x64 using the NVDA screen reader.

  • The website's favicon is no longer read by the NVDA screen reader.
  • The "Warning Icon" is no longer read by NVDA screen reader and the "Breached website" is read instead.
Status: RESOLVED → VERIFIED
Component: Password Manager → about:logins
Product: Toolkit → Firefox
Target Milestone: mozilla71 → Firefox 71
Version: unspecified → 70 Branch

@Jared, I have verified this issue on the Firefox Beta 70.0b13 (Build ID: 20191007220302) on Windows 10 x64 using the NVDA screen reader and it seems to be only partially fixed.

  • The website’s favicon in no longer read by the NVDA screen reader, but the “Warning icon” is still read by the NVDA screen reader.
  • This issue is fixed on the latest Nightly 71.0a1 ( Build ID: 20191007215350) and the “Warning icon” is not read by the NVDA screen reader.
Flags: needinfo?(jaws)

(In reply to Simona Rosu from comment #7)

@Jared, I have verified this issue on the Firefox Beta 70.0b13 (Build ID: 20191007220302) on Windows 10 x64 using the NVDA screen reader and it seems to be only partially fixed.

  • The website’s favicon in no longer read by the NVDA screen reader, but the “Warning icon” is still read by the NVDA screen reader.
  • This issue is fixed on the latest Nightly 71.0a1 ( Build ID: 20191007215350) and the “Warning icon” is not read by the NVDA screen reader.

This is expected since removing the 'alt' string from the Fluent file is considered changing a string and we can't change strings on Beta.

Flags: needinfo?(jaws)

@Jared, thanks for clarification. I'll change the status as verified then.

You need to log in before you can comment on or make changes to this bug.