Issue metadata
Sign in to add a comment
|
Disabled md-select is too light |
||||||||||||||||||||||
Issue descriptionThe text color of a disabled md-select is effectively rgb(208, 208, 208), which is the same as saying 18% opaque. This is a polish bug but also wouldn't meet contrast requirements. The md-select <select> is --secondary-text-color and also opacity: 0.33. Should only one of those styles be applied instead of compounding them?
,
Oct 20 2016
@michaelpg: I think that I tried to replicate exactly the disabled look of the paper-dropdown-menu at https://codereview.chromium.org/2406993002. See attachments comparing the two (they look the same level of gray to me). Are you saying that md-select does not match paper-dropdown-menu, or that regardless if it matches or not, it should be tweaked?
,
Oct 21 2016
,
Oct 31 2016
Ping @michaelpg, see comment #2.
,
Nov 1 2016
Regardless of whether it matches, I suspect it fails our contrast ratio a11y requirements. I also recall doing a comparison at the time and finding the md-select to be lighter than the old settings-dropdown-menu. I can't do that now without going back to an earlier build, but dropping in a <paper-dropdown-menu> in inspector shows it to be darker than the md-select (may or may not be a valid test).
,
Nov 1 2016
>Regardless of whether it matches, I suspect it fails our contrast ratio a11y requirements. Can we get UX or a11y opinion on this before changing it?
,
Nov 2 2016
lpalmaro: Do we need to worry about contrast ratio for disabled controls?
Some context found in issue 603620#1:
> All text on all pages must meet an absolute min contrast level of 4.5:1
But WCAG 2.0 actually says that "incidental" text ("part of an inactive user interface component") has no contrast requirement. Still, the effective contrast here if the values in my original description are correct is 1.54:1 which is really low.
http://webaim.org/resources/contrastchecker/?fcolor=d0d0d0&bcolor=ffffff
,
Dec 8 2016
,
Dec 13 2016
Any update? How can we help to ensure that these accessibility items are fixed for md-settings? Thanks!
,
Jan 26 2017
,
Feb 9 2017
,
Mar 16 2017
,
Mar 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ff04fe72fb9f627f2aae234a7893f88f2debbf71 commit ff04fe72fb9f627f2aae234a7893f88f2debbf71 Author: scottchen <scottchen@chromium.org> Date: Thu Mar 16 19:43:45 2017 MD Settings: change disabled select visuals. The visual we had for disabled md-select was too light for a11y standards; this CL gives it the same visuals as other disabled elements in md-settings. BUG= 657647 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2753983002 Cr-Commit-Position: refs/heads/master@{#457522} [modify] https://crrev.com/ff04fe72fb9f627f2aae234a7893f88f2debbf71/ui/webui/resources/html/md_select_css.html
,
Mar 16 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dpa...@chromium.org
, Oct 19 2016