MD Settings: change disabled select visuals.

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}
Committed: https://chromium.googlesource.com/chromium/src/+/ff04fe72fb9f627f2aae234a7893f88f2debbf71

[scottchen, 2017-03-16 00:54:53]

Description was changed from

==========
MD Settings: change disabled select visuals.

BUG=657647
==========

to

==========
MD Settings: change disabled select visuals.

BUG=657647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[scottchen, 2017-03-16 01:08:12]

Description was changed from

==========
MD Settings: change disabled select visuals.

BUG=657647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
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
==========

[scottchen, 2017-03-16 01:08:50]

scottchen@chromium.org changed reviewers:
+ hcarmona@chromium.org, michaelpg@chromium.org

[scottchen, 2017-03-16 01:08:51]

screenshot for before/after: http://imgur.com/a/oUVgG

[michaelpg, 2017-03-16 01:12:50]

lgtm

[hcarmona, 2017-03-16 16:45:55]

lgtm (I don't think you needed 2 reviewers for this change BTW)

[scottchen, 2017-03-16 17:54:24]

The CQ bit was checked by scottchen@chromium.org

[commit-bot: I haz the power, 2017-03-16 17:55:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottchen, 2017-03-16 17:55:15]

On 2017/03/16 16:45:55, hcarmona wrote:
> lgtm (I don't think you needed 2 reviewers for this change BTW)

Thanks! There was a big and long debate in the bug with a lot of people chiming
in, so I thought I'd get at least the original reporter and a11y expert to lg :)

[hcarmona, 2017-03-16 17:56:34]

On 2017/03/16 17:55:15, scottchen wrote:
> On 2017/03/16 16:45:55, hcarmona wrote:
> > lgtm (I don't think you needed 2 reviewers for this change BTW)
> 
> Thanks! There was a big and long debate in the bug with a lot of people
chiming
> in, so I thought I'd get at least the original reporter and a11y expert to lg
:)

👍

[commit-bot: I haz the power, 2017-03-16 19:43:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489686864733130, "parent_rev":
"4cf8e7e6215f686f727736826109303c3b115eec", "commit_rev":
"ff04fe72fb9f627f2aae234a7893f88f2debbf71"}

[commit-bot: I haz the power, 2017-03-16 19:44:42]

Description was changed from

==========
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
==========

to

==========
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}
Committed:
https://chromium.googlesource.com/chromium/src/+/ff04fe72fb9f627f2aae234a7893...
==========

[commit-bot: I haz the power, 2017-03-16 19:44:43]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ff04fe72fb9f627f2aae234a7893...