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

Issue 617712 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 671375



Sign in to add a comment

MD Settings: Customize font settings padding looks incorrect

Project Member Reported by michae...@chromium.org, Jun 6 2016

Issue description

Are the font settings and language settings properly aligned?

It looks to me like there is too much left padding on the controls:

* Font dropdowns and samples
* Language list and checkboxes

Should the controls be aligned with the subpage header, as in Manage Passwords? Or are the card previews out of date in that regard?
 
fonts.png
85.0 KB View Download
languages.png
58.1 KB View Download
Owner: michae...@chromium.org
Status: Assigned (was: Untriaged)
The elements in question should have a 56px indent, described here
https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/specs#%2FSPEC-settings_structure-subpage.png%3Fz=width


Screen Shot 2016-06-06 at 5.42.42 PM.png
62.4 KB View Download
Cc: dbeam@chromium.org michae...@chromium.org
Labels: -Pri-3 Pri-2
Owner: ----
Status: Available (was: Assigned)
Thanks. The cumulative padding is definitely more than that in these places and maybe others.

This is a visual polish/consistency issue that should be addressed before launch but probably not a dev blocker.
Labels: Hotlist-MD-Settings-General
Blocking: 671375
Owner: dschuyler@chromium.org
Status: Assigned (was: Available)
Summary: MD Settings: Customize font settings padding looks incorrect (was: MD Settings: Font & language settings padding looks incorrect)
This is no longer an issue with the new Languages section.

It sounds like the padding in Customize fonts should be 56px total instead of it's current 76px total.
Status: Started (was: Assigned)
Project Member

Comment 7 by bugdroid1@chromium.org, Mar 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/566e65ab665604104285fc725c65753423b1cb92

commit 566e65ab665604104285fc725c65753423b1cb92
Author: dschuyler <dschuyler@chromium.org>
Date: Thu Mar 16 00:24:10 2017

[MD settings] move font controls in MD settings

This CL outdents some UI controlls to place them in the correct positions
(they were indented too far). Also corrects some div tags.

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

Review-Url: https://codereview.chromium.org/2751063003
Cr-Commit-Position: refs/heads/master@{#457280}

[modify] https://crrev.com/566e65ab665604104285fc725c65753423b1cb92/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html

Status: Fixed (was: Started)

Sign in to add a comment