Closed Bug 1821788 Opened 1 year ago Closed 1 year ago

Link styles relying on use of color only


(Toolkit :: Themes, defect, P3)




114 Branch
Accessibility Severity s2
Tracking Status
firefox114 --- fixed


(Reporter: jules, Assigned: mseibert, Mentored)


(Blocks 1 open bug)


(Keywords: access, Whiteboard: [recomp])


(2 files)

We're setting text-decoration: none on the .text-link classes in common-shared.css and browser-shared.css. common-shared.css is also applying this rule to html|a. activity-stream.css is doing the same thing under their link rules.

By setting this rule we're failing WCAG Success Criterion 1.4.1 Use of Color. The underline is applied to links so they can be easily identified, specially for folks who cannot perceive color differences. See Failure of Success Criterion 1.4.1. Ideally, we should remove that rule from our links.

Whiteboard: [fidefe-reusable-components]
Keywords: access

please assign this to me

would the changes be done in all, common-shared.css files or a particular one

This is currently not an actionable bug and requires discussion first before making these changes

Asa says this would be nice to do over Slack, and mentioned it should at least respect the "Underline links" option in the Colours dialog, but I wasn't able to get that checkbox to do anything. We should probably file a ticket for that checkbox too.

Severity: -- → S3
Priority: -- → P3
Component: XUL Widgets → Themes

I looked into this a bit more - it seems the code related to the "Underline links" option in the colors dialog is still in place and working, but functions to provide more of a fallback/default for link styling, rather than forcing link underlining. You can see in the attached screenshot that the related styles can and do get overridden just about everywhere.

I'm going to hold off on filing a new bug for now, since a possible solution here would be to introduce some kind of "Always underline links" option that would function as more of an override than a fallback. It sounds like reworking the colors dialog is already on the accessibility team's radar.

Update: We contacted accessibility for support on this work and had a conversation with Asa. We are in agreement that underlines should be on the default link appearance. No historical reason why they were removed as it seems it was a cosmetic choice. We'll be talking to other engineers and planning the work around this since links appear throughout all of Firefox, and not just one stylesheet is responsible for this.

Whiteboard: [fidefe-reusable-components] → [fidefe-reusable-components][access-s2]

The severity field for this bug is set to S3. However, the accessibility severity is higher, [access-s2].
:dao, could you consider increasing the severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(dao+bmo)
Severity: S3 → S2
Flags: needinfo?(dao+bmo)
Assignee: nobody → mseibert
Mentor: dao+bmo
Pushed by
Link styles relying on use of color only. r=dao
Blocks: 1828430
Blocks: 1828433
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
See Also: → 1832855
Blocks: 1832855
See Also: 1832855
Accessibility Severity: --- → s2
Whiteboard: [fidefe-reusable-components][access-s2] → [fidefe-reusable-components]
Whiteboard: [fidefe-reusable-components] → [recomp]
You need to log in before you can comment on or make changes to this bug.