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

Issue 679174 link

Starred by 4 users
Status: Fixed
Owner:
lgrey@chromium.org
Closed: Jan 2017
Cc:
rdevlin....@chromium.org
nyerramilli@chromium.org
tapted@chromium.org
lgrey@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
mac-bugs-priority@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Extensions] Adding new extension causes icon overlap.

Project Member Reported by apaci...@chromium.org, Jan 7 2017 
Chrome Version: 57.0.2975.0
OS: Mac 10.11.6

Repros 50% of the time.

What steps will reproduce the problem?
(1) Have some extensions in the toolbar.
(2) Add a new extension from the Chrome Web Store.

(3) (Optional) Manually move some icons around on the visible toolbar.

What is the expected result?
New extension icon appears on toolbar without overlap.

What happens instead?
Left-most extension overlaps the bookmark "star" icon.
Screen Shot 2017-01-07 at 15.34.39.png
20.8 KB View Download
Screen Shot 2017-01-07 at 15.36.46.png
21.8 KB View Download

Comment 1 by meh...@chromium.org, Jan 8 2017

Labels: -Type-Bug -Pri-2 Pri-1 Type-Bug-Regression
The overlapping is also happening, when you already have extension icons on the Toolbar and starting Chrome (Canary).

Comment 2 by tapted@chromium.org, Jan 9 2017

Cc: tapted@chromium.org rdevlin....@chromium.org
Labels: ReleaseBlock-Beta
Owner: lgrey@chromium.org
Probably r441925 -> https://codereview.chromium.org/2607533004 "[Mac] Flip toolbar in RTL"

Comment 3 by lgrey@chromium.org, Jan 9 2017

Cc: lgrey@chromium.org msrchandra@chromium.org ranjitkan@chromium.org nyerramilli@chromium.org
 Issue 679249  has been merged into this issue.

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

Status: Started (was: Assigned)

Comment 5 by tapted@chromium.org, Jan 19 2017

Labels: M-57
Status: Fixed (was: Started)
This was reverted in r442257 and relanded, fixed in r443345. See  Issue 648563 . So I think this can be closed. I haven't noticed anything strange.
Project Member

Comment 6 by blocker...@chrome-infra-auth.iam.gserviceaccount.com, Jan 19 2017

Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-57; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-57 label, otherwise remove Merge-TBD label. Thanks.

Comment 7 by tapted@chromium.org, Jan 19 2017

Labels: -Merge-TBD
m57 hasn't branched.
►

Sign in to add a comment