Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
toolbar.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Anonymous (not verified)
Created:
6 Jul 2013 at 16:15 UTC
Updated:
9 Jan 2019 at 23:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedAttached is a fix that adheres to AAA compliance with WCAG.
Comment #2
mgiffordWhy did you go to AAA rather than just AA? According to
http://webaim.org/resources/contrastchecker/
#707070 is fine against #fff
If we can meet AAA, great, but that hasn't been our goal.
Comment #3
mgifford#1: drupal.toolbar_submenu_color_contrast_2036239_1.patch queued for re-testing.
Comment #5
mgiffordReroll
Comment #6
mgifford#5: drupal.toolbar_submenu_color_contrast_2036239_5.patch queued for re-testing.
Comment #7
corbacho commentedThe patch applies nicely.
Almost no difference, but I think it's true that is more readable with that extra bit of contrast.
And still the bold font-weight of the :active element is very noticeable.
Notice that to achieve the AAA (7:1 contrast) would have been enough #595959 against the background #ffffff
Comment #8
mgiffordExcellent, thanks for the nice, clear comparison!
Comment #9
alexpottCommitted a2a4e8f and pushed to 8.x. Thanks!
Comment #10
mgiffordJust tagging. Great that this is already fixed in Core!