New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 606072 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Theme color on Android allows sites to de-emphasize the lock icon and make "https" gray.

Project Member Reported by lgar...@chromium.org, Apr 22 2016

Issue description

<b>Version: <Kenneth, what is the frequency?></b>
<b>OS: <please tell me it's not XP></b>

What steps will reproduce the problem?
(1) Visit github.com
(2) Visit github.com/lgarron/badssl.com

What is the expected output?
The lock icon stays the same and "https" stays green. (github-no-theme.png)

What do you see instead?
The lock icon is replaced by a smaller one (with white padding around it) and the scheme becomes gray. (github-fdfdfd.png)

GitHub sends <meta name="theme-color" content="#f5f5f5"> to Android, which triggers Hera coloring. It *used* to be that we would show the usual lock icon and green scheme for light theme-colors.
For dark colors, we always showed the inverted lock icon and gray scheme, but that didn't look excellent for all colors (see  Issue 436553 ). At some point, it seems we switched to inverted at all times.
Was this intentional?
Given that their theme color matches the URL bar color, GitHub seems to be doing it purely to reduce the visibility of our security indicators. Are we okay with that?

yusufo@: Do you know how this decision was reached?
felt@, palmer@, ainslie@: Bug? WontFix? "Urgh"?
 
default.png
46.2 KB View Download
fdfdfd.png
46.5 KB View Download
ffffff.png
44.8 KB View Download
github-no-theme.png
718 KB View Download
github-fdfdfd.png
161 KB View Download
Cc: maxwalker@chromium.org emilyschechter@chromium.org
Max's new icons won't use the white background rect and so the lock will be the same size in both cases. 

Comment 2 by yus...@chromium.org, Apr 25 2016

We have mainly done this to get out of situations where green emphasis was getting lost inside some colors that has enough tinge of green in them. And the main thing that we wanted to stay careful about was theme color disappearing on broken lock and red emphasis staying there.

Am I misremembering this, or the new lock icons were going to be a single color irrespective of the SSL state?
The lock color will change in some states. It'll still be green with the normal gray toolbar and then black/white to match the toolbar-controls + omnibox text for theme-color toolbars. A white lock icon example (outdated mock, not a spec) is attached.  

https://docs.google.com/presentation/d/1symd8YrF_GF3KG7gEaRgpl9xHogxAqD3P7dVojLZwT4/edit#slide=id.gf2f47636d_0_5
white-locks-theme-color.png
752 KB View Download
I guess this is WontFix, then?

I just came across someone who was confused about the current behaviour: "What does it mean for smaller green lock icon ?" [1]
sealhouse.cc does the same as GitHub.

[1] https://productforums.google.com/forum/?utm_medium=email&utm_source=footer#!msg/chrome/xRgn5M33BPs/F9Jg4dprAwAJ

Comment 5 Deleted

Comment 6 by f...@chromium.org, Jul 19 2016

Status: Fixed (was: Untriaged)
It looks like this has been fixed.
Components: -Security>UX
Labels: Team-Security-UX
Security>UX component is deprecated in favor of the Team-Security-UX label

Sign in to add a comment