#44763 closed defect (bug) (fixed)
Custom CSS line number color contrast is too low
Reported by: | earnjam | Owned by: | johnbillion |
---|---|---|---|
Milestone: | 5.0 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | Customize | Keywords: | |
Focuses: | accessibility | Cc: |
Description
This came up in Gutenberg audit of the Custom HTML block and I couldn't find a core ticket about it.
https://github.com/WordPress/gutenberg/issues/8736
The color contrast of the line numbers in the Additional CSS control in the Customizer is currently 2.66. We should be above 4.5:1 to meet WCAG AA level.
Attachments (2)
Change History (16)
#5
@
6 years ago
@abdullahramzan the patch looks good to me, thanks! (but please don't forget we try to leave good-first-bug
labelled issues to engage new contributors to learn how to contribute to WordPress Core :)
By the way, thank you for this quick patch :)
#6
@
6 years ago
I checked this out at the WordCamp Brighton contributor day and the patch works fine.
#7
@
6 years ago
- Owner set to johnbillion
- Resolution set to fixed
- Status changed from new to closed
In 43574:
#8
@
6 years ago
- Keywords fixed-major added; good-first-bug has-screenshots has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
Note: See
TracTickets for help on using
tickets.
Hello and thanks for your Trac Ticket.
Nice catch.
Indeed, the color contrast is not compliant with AA criteria.
I made some checks/tests:
#F7F7F7
for the background color of the gutter, we need at least#666
for line numbers..min.css
files, and to edit both LTR and RTL versions.wp-admin/css/code-editor.css
.I guess the following CSS rule should do the job:
.wrap .CodeMirror .CodeMirror-linenumber { color: #666; }
A great
good-first-bug
for potentially new contributors :)Cheers,
Jb