[MDL-71385] Colour contrast issues with striped tables and dimmed text - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.9.6, 3.10.6, 3.11.2, 4.0
Fix Version/s: 3.10.7, 3.11.3
Component/s: Accessibility
Labels:

Affected Branches:
MOODLE_310_STABLE, MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
Fixed Branches:
MOODLE_310_STABLE, MOODLE_311_STABLE
Pull from Repository:
git://github.com/junpataleta/moodle.git
Pull 3.11 Branch:
~~MDL-71385~~-311-3
Pull 3.11 Diff URL:
https://github.com/junpataleta/moodle/compare/7fdf72a268...MDL-71385-311-3
Pull Master Branch:
~~MDL-71385~~-master-3
Pull Master Diff URL:
https://github.com/junpataleta/moodle/compare/443a980aea...MDL-71385-master-3
Testing Instructions:
Hide

Test on Boost and Classic themes

Log in as an admin

Go to Site administration / Plugins / Enrolments /Manage enrol plugins

Run axe DevTools or WAVE

Confirm that there are no colour contrast issues on the page, especially on the table.
Show
Test on Boost and Classic themes Log in as an admin Go to Site administration / Plugins / Enrolments /Manage enrol plugins Run axe DevTools or WAVE Confirm that there are no colour contrast issues on the page, especially on the table.

Description

When text-muted or dimmed_text class is applied to a text in a striped table, the colour contrast ratio between the greyed-background rows with the dimmed text is 4.19:1 which breaks WCAG 2.1 Level AA requirement.

Either we darken the .text-muted and .dimmed_text classes or lighten the background of the striped table or both to achieve the required 4.5:1 contrast ratio.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

after patch.png
60 kB
06/Aug/21 12:26 PM
patched-v2.png
47 kB
16/Aug/21 6:47 PM
sample-striped-table.png
50 kB
20/Apr/21 9:41 PM
test_after.png
87 kB
09/Sep/21 5:37 PM
test_before.png
108 kB
09/Sep/21 5:37 PM

Issue Links

Discovered while testing

MDL-71017 Add the ability to configure OAuth2 services for login only; add login display name

Closed

Activity

People

Assignee:: Jun Pataleta

Reporter:: Jun Pataleta

Peer reviewer:: Michael Hawkins

Integrator:: Eloy Lafuente (stronk7)

Tester:: Eloy Lafuente (stronk7)

Participants:: CiBoT, Eloy Lafuente (stronk7), Jun Pataleta, Michael Hawkins

Component watchers:

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Dates

Created:: 20/Apr/21 9:40 PM

Updated:: 11/Sep/21 6:50 PM

Resolved:: 11/Sep/21 6:50 PM

Fix Release Date:: 13/Sep/21

Time Tracking

Estimated:

Not Specified

Remaining:

Logged:

6h 30m