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

Issue 679427 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

No separator should appear at the right-hand side of the shelf

Project Member Reported by sgabr...@chromium.org, Jan 9 2017

Issue description

An extra separator sometimes appears on the right-hand side of the shelf as shown in the screenshot attached.
 
Screen Shot 2017-01-09 at 11.09.22 AM.png
79.8 KB View Download
Is this on canary or beta channel?
Cc: yiyix@chromium.org
yiyix@, could it be your recent change?
#1, I can see it in dev - 57.0.2970.0 on samus.

Comment 4 by yiyix@chromium.org, Jan 9 2017

Owner: yiyix@chromium.org
It is due to my changes. Sorry i didn't notice it when i test the UI. 

Comment 5 by yiyix@chromium.org, Jan 9 2017

This bug is due to changes in this, https://bugs.chromium.org/p/chromium/issues/detail?id=657211,
and it had made to M56. I will fix it and request to merge to M56. 
FYI https://bugs.chromium.org/p/chromium/issues/detail?id=657211 hasn't been merged to M56 yet.  We can wait until this fix is ready and merge both together.

Comment 7 by yiyix@chromium.org, Jan 9 2017

@sgabriel, could you please correct the bug description? This bug should always appear.
Summary: No separator should appear at the right-hand side of the shelf (was: Extra separator appears on the right side of the shelf)
Note that I see this consistently on dev channel in Guest mode.
Labels: -Type-Bug -Pri-2 Proj-MaterialDesign-CrOS ReleaseBlock-Stable Pri-1 Type-Bug-Regression
Status: Started (was: Assigned)
Issue 677200 has been merged into this issue.

Comment 13 by yiyix@chromium.org, Jan 10 2017

Labels: Merge-Request-56
Project Member

Comment 14 by sheriffbot@chromium.org, Jan 11 2017

Labels: -Merge-Request-56 Hotlist-Merge-Approved Merge-Approved-56
Your change meets the bar and is auto-approved for M56. Please go ahead and merge the CL manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Started)
Project Member

Comment 16 by bugdroid1@chromium.org, Jan 11 2017

Status: Verified (was: Fixed)
Verified on ChromeOS 9202.0.0, 57.0.2984.0
Also verified on 9000.66.0, 56.0.2924.67

Sign in to add a comment