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

Issue 612615 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: May 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocking:
issue 547953



Sign in to add a comment

[Mac] [Material Design] Bookmarks Bar: The chevron icon is too large (or the hover state of the chevron icon is too large)

Project Member Reported by meh...@chromium.org, May 17 2016

Issue description

Version: Chromium Build 394171
OS: 10.11.5


What steps will reproduce the problem?
(1) save a lot of bookmarks, so that the chevron icon appears


What is the expected output? What do you see instead?
The chevron icon is too large. Is the icon may be a 2X for retina? If the icon size is correct, then the hover state is much larger than the hover state of the bookmarks.

Please use labels and text to provide additional information.
A screenshot is attached.

Thanks,
Mehmet

 
Bildschirmfoto 2016-05-18 um 00.30.52.png
15.7 KB View Download

Comment 1 by shrike@chromium.org, May 17 2016

Blocking: 547953
I have it 16x16 but it should be 8x8.

Comment 2 by shrike@chromium.org, May 18 2016

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, May 18 2016

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

commit d77c14d86100cbf91869f78ab6361eab56de1ca6
Author: shrike <shrike@chromium.org>
Date: Wed May 18 18:50:56 2016

[Mac][Material Design] Change bookmark chevron icon size to 8x8.

The chevron icon is currently 16x16, which is larger than it should be.

R=avi@chromium.org
BUG= 612615 

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

[modify] https://crrev.com/d77c14d86100cbf91869f78ab6361eab56de1ca6/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.mm

Comment 4 by shrike@chromium.org, May 18 2016

Status: Fixed (was: Started)

Sign in to add a comment