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

Issue 646975 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

clock looks weird when shelf is vertically aligned.

Project Member Reported by yiyix@chromium.org, Sep 14 2016

Issue description

Version: M54
OS: Chrome OS

What steps will reproduce the problem?
(1) set ash-md to experimental
(2) change shelf alignment to vertically aligned

What is the expected output?
The time can be properly seen.

What do you see instead?
The time cannot be properly seen. (refer to the picture)

Please use labels and text to provide additional information.

 
Screenshot from 2016-09-14 15:12:11.png
4.5 KB View Download
Labels: M-55
Status: Available (was: Untriaged)
Probably due to Ben/Evan recently increasing the font size used for MD. This likely just requires a small change to the layout manager used by the time view.

Comment 2 by yiyix@chromium.org, Oct 5 2016

Owner: yiyix@chromium.org
Status: Started (was: Available)

Comment 3 by yiyix@chromium.org, Oct 5 2016

I fixed it while working on updating spacing between items in the system tray area.
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 6 2016

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

commit d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f
Author: yiyix <yiyix@chromium.org>
Date: Thu Oct 06 01:00:24 2016

Update spacing between material design icons in the system tray

Updated spacing between material design icons to 6dp; adjusted the
spacing between the hours and minutes when the clock is aligned
vertically.

TEST=MANUAL
- Measured the distance between tray items using gimp while setting
  the shelf alignment to bottom, right and left.

BUG=630296,  646975 

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

[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/date/date_view.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_constants.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_constants.h
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_utils.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/user/tray_user.cc

Yi, does the CL in #4 fix this issue?
Status: Fixed (was: Started)
Verified fixed on ToT.
Status: Verified (was: Fixed)
Verified on ChromeOS 8872.16.0, 55.0.2883.20
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f

commit d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f
Author: yiyix <yiyix@chromium.org>
Date: Thu Oct 06 01:00:24 2016

Update spacing between material design icons in the system tray

Updated spacing between material design icons to 6dp; adjusted the
spacing between the hours and minutes when the clock is aligned
vertically.

TEST=MANUAL
- Measured the distance between tray items using gimp while setting
  the shelf alignment to bottom, right and left.

BUG=630296,  646975 

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

[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/date/date_view.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_constants.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_constants.h
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/tray/tray_utils.cc
[modify] https://crrev.com/d73de42d6da01d9c7c24c0b6573e89ce6bab3a3f/ash/common/system/user/tray_user.cc

Comment 9 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment