Closed Bug 255726 Opened 20 years ago Closed 16 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
http://hg.mozilla.org/mozilla-central/rev/177e0a27bd42
Status: ASSIGNED → RESOLVED
Closed: 16 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: