contrast between link and non-link text in webstaff grids

Bug #1756572 reported by Galen Charlton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.12
Fix Released
Medium
Unassigned
3.0
Fix Released
Medium
Unassigned

Bug Description

As a result of the patch for bug 1746365, the contrast between the new link color and the background of selected rows in grids is now 4.71:1, achieving compliance WCAG 2.0 AA guidelines.

However, the color of non-link text (#333) and link text (#286090) has a contrast ratio of only 1.90:1, meaning that we're not achieving the 3:1 ratio suggested when doing nothing else to distinguish link text from non-link text (see https://webaim.org/blog/wcag-2-0-and-link-colors/).

Evergreen master

Revision history for this message
Galen Charlton (gmc) wrote :

(In retrospect, I think I may have jumped the gun on merging bug 1746365).

Anyway, changing the non-link text color to #000 (black) in grid cells would bring the contrast to 3.17:1, which meets the suggested ratio. (And looking at it more, changing the default text color from Bootstrap's default of #333 to #000 across the board might be a good idea in general).

The branch user/gmcharlt/lp1756572_wip has a change that does some of this as a discussion piece; it also adds underlines for links inside grid contents.

Kathy Lussier (klussier)
Changed in evergreen:
assignee: nobody → Kathy Lussier (klussier)
Revision history for this message
Kathy Lussier (klussier) wrote :

I this this looks good. I generally like underlined links because it's more clear to the user that they are links. I can add a signoff, but were you looking to get more feedback before merging?

Changed in evergreen:
assignee: Kathy Lussier (klussier) → nobody
Revision history for this message
Galen Charlton (gmc) wrote :

On reflection, I think the patch can be signed off as is; while there are other places in the default Bootstrap CSS where text defaults to #333, e.g., in buttons, the patch as is offers an improvement.

Revision history for this message
Kathy Lussier (klussier) wrote :

Thanks Galen! I've merged the improvement to master, release 3.0 and 2.12.

Changed in evergreen:
status: New → Fix Committed
milestone: none → 3.1-rc
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.