1522207 - [meta] Full page colour contrast audit

Status: RESOLVED FIXED

(DevTools :: Accessibility Tools, enhancement)

Description

[Yura Zenevich [:yzen] (please use "needinfo")]

In addition to providing colour contrast information for individual nodes/accessible objects, a11y panel should also have a display mode that shows all colour contrast accessibility issues on the page.

Comment 3

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: UX Spec to help with ui review.

Comment 4

[Victoria Wang [:victoria]]

Comment on attachment 9059165 [details]
UX Spec to help with ui review.

Hi Yura,

This looks great, and is so exciting!

The only thing I noticed that seemed odd is how the contrast badges inside the tree work the same way as the top one.  I would expect the node badges to do some action on the node only. I'd suggest keeping just the toolbar badge as a functional filter button, and make the other badges the unclickable (no background) style.

Comment 5

[Julien Cristau [:jcristau] (back April 22)]

Is this something we should mention in the 68 release notes? If yes can you suggest some wording?

Comment 6

[Yura Zenevich [:yzen] (please use "needinfo")]

(In reply to Julien Cristau [:jcristau] from comment #5)

Is this something we should mention in the 68 release notes? If yes can you suggest some wording?

It's in Firefox and Runtime Visuals trello boards. I'm not sure if things that are marked as ready to land there get into release notes automatically, if not, hopefully, I can defer that to Martin or Asa.

Comment 7

[Asa Dotzler [:asa]]

Yes, this should be a part of the release notes.

Comment 8

[Asa Dotzler [:asa]]

Release Note Request (optional, but appreciated)
[Why is this notable]:
[Affects Firefox for Android]:
[Suggested wording]:
[Links (documentation, blog post, etc)]:

Potential Release Note:

In addition to providing color contrast information for individual elements of a page, Firefox now offers a full page color contrast audit that identifies all element on a page that fail color contrast checks.

Comment 9

[Martin Balfanz [:mbalfanz]]

Definitely relevant for the release notes, though I was also under the impression that features tracked in the R&V trello board are tracked for relnotes automatically 🤔

I'll clarify with Thomas for the next time.

Comment 10

[Julien Cristau [:jcristau] (back April 22)]

Thanks Asa, added to the beta release notes.

FWIW this is not automatic. If something is in the Firefox (not R&V) trello board it'll be picked up (which is how I came here), but I prefer to ask the people who know what the feature is about for some wording and/or relevant links, hence this request here.

Comment 11

[Yura Zenevich [:yzen] (please use "needinfo")]

Closed since follow up work will happen in individual bugs.

Comment 12

[Irene Smith]

This is tracked on GitHub in MDN/Sprints #1772

Comment 13

[Irene Smith]

Added a section showing the check for contrast issues with the comment that more accessibility problems will be added in the future.

Accessibility Inspector Find contrast issues

Please let me know if this is enough for now.

This weeks tool tip for the Newsletter will cover this new feature

The Accessibility Inspector page as a whole needs an update, but perhaps when we start documenting 69?

Comment 14

[Yura Zenevich [:yzen] (please use "needinfo")]

(In reply to Irene Smith from comment #13)

Added a section showing the check for contrast issues with the comment that more accessibility problems will be added in the future.

Accessibility Inspector Find contrast issues

Please let me know if this is enough for now.

This weeks tool tip for the Newsletter will cover this new feature

The Accessibility Inspector page as a whole needs an update, but perhaps when we start documenting 69?
Yeah that would be fantastic.