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

Issue metadata

Status: Verified
Owner:
Closed: Jul 2010
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug
M-6

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 49996: Edit and Zoom on the wrench menu are highlighted ( Bold )

Reported by srikanthk@chromium.org, Jul 22 2010 Project Member

Issue description

Chromium	6.0.475.0 (Developer Build 53384)
WebKit	534.3
V8	2.3.2
User Agent	Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; en-US) AppleWebKit/534.3 (KHTML, like Gecko) Chrome/6.0.475.0 Safari/534.3
Command Line	 /Applications/Chromium.app/Contents/MacOS/Chromium -psn_0_11049609

What steps will reproduce the problem?
1. Launch chrome click on the wrench menu.
2. Check the Zoom and Edit .

What is the expected output?
Should not be highlighted ( Bold )

What do you see instead?
Zoom and Edit are bold highlighted.
 
Picture 5.png
55.6 KB View Download

Comment 1 by meh...@chromium.org, Jul 23 2010

Yes I noticed it also. It looks not consistently with the other items. 

See  Issue 49350  . But it was marked as "won't fix" :-(

Comment 2 by mega...@gmail.com, Jul 23 2010

On Windows it looks consistently not bold.  Why is it bold on Mac?  I don't know about Mac but on Windows bold menu items are default menu items... only applicable to context menus where left clicking (possibly double clicking) whatever you right clicked does the default action.

It does help indicate they aren't normal menu items but there might be a better way to do it than bolding.  Maybe graying out?

Comment 3 by rsesek@chromium.org, Jul 23 2010

Labels: UI-Needed
In the mocks for this menu, they were bold. Last I checked, they were also bold on Linux. A UI lead should probably weigh-in here.

Comment 4 by jrg@chromium.org, Jul 23 2010

Labels: Mstone-6 ReleaseBlock-Beta
Status: Assigned

Comment 5 by rsesek@chromium.org, Jul 26 2010

Labels: -UI-Needed
Status: Started
Got the OK for doing this during the M6 go/no-go decisions.

Comment 6 by dhw@chromium.org, Jul 26 2010

 Issue 49350  has been merged into this issue.

Comment 7 by bugdro...@gmail.com, Jul 26 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=53638 

------------------------------------------------------------------------
r53638 | rsesek@chromium.org | 2010-07-26 10:53:04 -0700 (Mon, 26 Jul 2010) | 6 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/nibs/Toolbar.xib?r1=53638&r2=53637

[Mac] In the Wrench menu, unbold the special item font and make it 13pt.

BUG= 49996 
TEST=Visual

Review URL: http://codereview.chromium.org/3029026
------------------------------------------------------------------------

Comment 8 by bugdro...@gmail.com, Jul 26 2010

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=53701 

------------------------------------------------------------------------
r53701 | rsesek@chromium.org | 2010-07-26 15:10:54 -0700 (Mon, 26 Jul 2010) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/472/src/chrome/app/nibs/Toolbar.xib?r1=53701&r2=53700

Merge 53638 - [Mac] In the Wrench menu, unbold the special item font and make it 13pt.

BUG= 49996 
TEST=Visual

Review URL: http://codereview.chromium.org/3029026

TBR=rsesek@chromium.org
Review URL: http://codereview.chromium.org/3066006
------------------------------------------------------------------------

Comment 9 by rsesek@chromium.org, Jul 26 2010

Status: Fixed

Comment 10 by bugdro...@gmail.com, Aug 2 2010

Labels: Verifier-Deepak
Verifier Updated by AutoAllocator, contact AmolK or KrisR for questions

Comment 11 by deepakg@chromium.org, Aug 4 2010

Status: Verified
Verified in 6.0.472.22 (Official Build 54852) dev

Comment 12 by bugdroid1@chromium.org, Oct 12 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 13 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Area-UI -Feature-Browser -Mstone-6 M-6 Cr-UI Cr-UI-Browser-Core

Comment 14 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment