Material Refresh new tab page: there's too much padding between the top site icon and its label
Reported by
billdill...@gmail.com,
Aug 31
|
|||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.72 Safari/537.36 Steps to reproduce the problem: There's too much padding between the top site icons and their text labels. The Android 8.0 start screen had this same issue and it was fixed in Android 9 (I'm using a 1st gen Pixel XL) What is the expected behavior? What went wrong? apologies if this is already fixed or a known issue Did this work before? N/A Chrome version: 69.0.3497.72 Channel: beta OS Version: 10.0 Flash Version:
,
Sep 2
,
Sep 3
Able to reproduce the issue on chrome reported version# 69.0.3497.72 and on latest chrome# 70.0.3538.0 using Mac 10.13.1, Windows-10 and Ubuntu 17.10. As this issue is seen from M-69(from introduction of #upcoming-ui-features flag), hence considering this issue as Non-Regression and marking it as Untriaged. Thanks!
,
Sep 4
,
Sep 4
Thank you for the report! This is a design decision for the new Material refresh and is WAI.
,
Sep 10
,
Sep 10
Joel - take a look?
,
Sep 10
,
Sep 10
WontFix -> Assigned
,
Sep 10
FYI: It seems that according to the installed theme, the spacing between the top site icon and its label differ. Please find enclosed a comparison between the Default Theme and https://chrome.google.com/webstore/detail/simple-blue-theme/lickmlblgejhimhgknmmmpannbigofjp?utm_source=chrome-ntp-icon on macOS.
,
Sep 18
I have no themes installed
,
Sep 20
,
Sep 20
,
Sep 20
The default padding is implemented as intended (based on core material spacing/margins). Added perceived spacing (when theme is installed) is due to a pill UI we draw behind the title to ensure text legibility w/ 3rd party themes.
,
Sep 26
,
Oct 9
,
Oct 23
,
Oct 24
Again, I have no themes installed. So if this isn't a 'bug', I guess it's just me asking the team to consider making a small design change/adjustment :)
,
Oct 24
We appreciate the request. I'll defer to c15 here: the padding is by design based on Material Design spacing recommendations. |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by pkasting@chromium.org
, Aug 31Components: -UI UI>Browser>NewTabPage
Labels: Proj-MdRefresh