Issue metadata
Sign in to add a comment
|
Security state icon alignment and sizing on Android |
||||||||||||||||||||||||
Issue descriptionFrom maxwalker: "there are still some issues around alignment, spacing and icon size. When testing on a Nexus 6P I noticed that the URL is too far on the left and the icon isn't centered within its bounding box. On the Pixel C the spacing and alignment are correct, however the icon seems to be too big." Splitting this out from https://bugs.chromium.org/p/chromium/issues/detail?id=626646.
,
Oct 19 2016
,
Nov 24 2016
,
Nov 25 2016
,
Nov 30 2016
,
Nov 30 2016
,
May 4 2017
I just realized this is still an issue, especially on Pixel C. tedchoc, do you know why the tablet layout appears to use a different layout for the omnibox?
,
May 4 2017
> tedchoc, do you know why the tablet layout appears to use a different layout for the omnibox? To be clear: I'm aware that we do things differently for the tablet form factor. But I'm wondering why the security indicator icon doesn't automatically show up in the same place relative to the text.
,
May 5 2017
the padding for all of this is likely from the material redesign of chrome for android, which is like 3ish years ago. If the designers think it should be updated, then I don't think there is any reason stopping us (probably new designers think differently than designers at the time it was originally decided upon). The main thing to consider is that we show icons in the omnibox dropdown on tablets, so the spacing between the security indicator and text is the same used for suggestions. I wouldn't be surprised if that was a motivating factor on increasing the spacing compared to phones (and tablets have more space to begin with so it might be that phones were more compressed to give more room to the url)
,
May 16 2017
,
May 16 2017
,
Nov 10 2017
,
Dec 1 2017
maxwalker could you confirm if this is still an issue please?
,
Dec 1 2017
I just checked. It looks good on both phones and tablets now. Please feel free to close the issue.
,
Dec 1 2017
No longer applicable, so closing this out. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by raymes@chromium.org
, Sep 6 2016