Closed Bug 255726 Opened 21 years ago Closed 17 years ago

about:config context menus do not have default options bolded

Categories

(Firefox :: Menus, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Firefox 3.1b1

People

(Reporter: richard.corbin, Assigned: asqueella)

Details

Attachments

(1 file, 1 obsolete file)

When right clicking an option in about:config screen, the default option (i.e. what would happen if user double-clicks) on that menu is not marked in bold. In all other cases in Windows and Firefox that I could find the default option is in bold. Reproduceable: Always Steps to Reproduce: 1. Enter about:config into location bar 2. right click any option (either boolean, integer, or string) Actual results: No context menu items are bold Expected Results: The default action (what would happen when double-clicked) should be highlighted as a bold option in the menu. ... So the bolded options dependant on type should be as follows: * Boolean: Toggle * String: Modify * Integer: Modify
Assignee: firefox → nobody
QA Contact: bugzilla → menus
In which other cases in Firefox do you see this?
Assignee: nobody → asqueella
Status: NEW → ASSIGNED
Attachment #208578 - Flags: review?(mconnor)
Attachment #208578 - Flags: review?(mconnor) → review?(gavin.sharp)
Comment on attachment 208578 [details] [diff] [review] add default="true" and move the default items to the top of menu Hmm, I'm kinda thinking this menu could use a separator or two, but that can probably be dealt with that separately. r=me on the addition of the default attribute, though I think you should get mconnor or beltzner to ui-review the position changing.
Attachment #208578 - Flags: review?(gavin.sharp) → review+
Attachment #208578 - Flags: ui-review?(beltzner)
Comment on attachment 208578 [details] [diff] [review] add default="true" and move the default items to the top of menu ui-r+, though gavin's right, a separator between the default action and the other values is needed.
Attachment #208578 - Flags: ui-review?(beltzner) → ui-review+
Keywords: checkin-needed
OS: Windows XP → All
Hardware: PC → All
Version: unspecified → Trunk
Attached patch updated patchSplinter Review
Attachment #208578 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: