Incognito Tab Hover and Active State Needs to be More Visible |
|||||||||||||
Issue descriptionMaterial Refresh Incognito contrast ratios should not be regressed from M67 Stable. M67 Stable: Active-Inactive = 1.47 Inactive-Inactive+Hovered = 1.4 M69 Material Refresh: Active-Inactive = 1.4 Inactive-Inactive+Hovered = 1.21
,
Jul 20
,
Jul 25
,
Jul 25
,
Jul 26
,
Jul 26
,
Jul 28
Any updates on this? This is an a11y blocker for us.
,
Jul 28
Rob's comment 6 is confusing; it says this bug is related to itself, and reassigns to Orin without a color spec from Alan. Addressing bug 856893 might change the hovered numbers (though I don't know to what), but if the active-vs.-inactive contrast ratios are too low, Alan needs to adjust the master spec of what the frame and active tab colors are.
,
Jul 28
I figured Rob meant 856893. I am trying to nail down the right course of action for both these bugs together (see my last comment on 856893) but my big concern at this point is that the whole new contrast-ratio-driven approach is too experimental for post-branch development. It may well be brilliant, and maybe we're close, but it's all new & different, and I suppose time is short. http://crrev.com/c/1147605 shows some progress, in 3 different directions: 1) interpolation within preferred range, 2) constants straight from an updated spec, and 3) the new contrast-ratio targeting approach. If we can get #3 going and it works as well as hoped, it might be great and handle all the cases, but it's uncertain at this point.
,
Aug 2
Okay, the contrast-ratio targeting approach is submitted to CQ so we should see it working in tomorrow's canary. What I think would be helpful in knocking out this issue is to: * State the process used to measure live contrast ratios (is it just screenshot + color tool in GIMP + contrast-ratio.com? A screenshot with measured pixels marked would be great.) * Post the updated contrast ratio values along with the raw colors used to compute them. I suspect we may end up tuning some constant values targeting specific contrast ratios. Right now they are definitely smaller than the 1.4 mentioned on #0.
,
Aug 2
I suspect the ratios in comment 0 were "contrast-ratio.com using the spec values" but I'm not sure.
,
Aug 3
Peter is correct about #0. Gonna look into Canary today.
,
Aug 3
Note, the contrast ratio approach in comment 10 hasn't landed yet because it failed browser tests.
,
Aug 4
Yeah, I am learning that I should say "look for it in Canary tomorrow" when CLs *land*, not when submitted to CQ. The failures appear to be unrelated to the change itself, but it's new to me and in any case will need to be troubleshot.
,
Aug 6
,
Aug 6
Friendly ping! Is this in today's canary?
,
Aug 6
No.
,
Aug 7
Now the CL is landed, so look for contrast-ratio-driven tab hover opacities in tomorrow's canary. My concern about the targeted numerical contrast ratio values (last paragraph of comment #10) still stands.
,
Aug 10
Implementation looks pretty good across default, incognito, colored title bars, and themes. I'd be interested in what some subtle tuning might look like to increase the contrst, but I don't think it's possible for M69 (per Mark's advice). Darker tabstrips could use a brighter base color, but increasing the contrast for darker tabstrips runs the risk of blending inactive-hover tabs with active tabs in the default tabstrip case. Possibly worth opening up another bug for?
,
Aug 11
Doing things like small contrast increases and color changes across all themes, including default, is pretty easy and safe, even for M69. My read on this bug is that this is about the default theme not having sufficient contrast. If you can come up with a tweak you want to make to the default theme for incognito, we can adapt it to work for other themes/settings.
,
Aug 21
,
Aug 23
Moving to 70 - Already received a11y signoff for M69.
,
Sep 13
,
Sep 20
,
Sep 26
,
Oct 11
,
Dec 11
|
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by robliao@chromium.org
, Jul 20Owner: bettes@chromium.org
Status: Assigned (was: Untriaged)