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)
Firefox
Menus
Tracking
()
RESOLVED
FIXED
Firefox 3.1b1
People
(Reporter: richard.corbin, Assigned: asqueella)
Details
Attachments
(1 file, 1 obsolete file)
2.18 KB,
patch
|
Details | Diff | Splinter Review |
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
Updated•20 years ago
|
Assignee: firefox → nobody
QA Contact: bugzilla → menus
Comment 1•20 years ago
|
||
In which other cases in Firefox do you see this?
Assignee | ||
Comment 2•20 years ago
|
||
Assignee | ||
Updated•19 years ago
|
Attachment #208578 -
Flags: review?(mconnor) → review?(gavin.sharp)
Comment 3•19 years ago
|
||
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+
Assignee | ||
Updated•19 years ago
|
Attachment #208578 -
Flags: ui-review?(beltzner)
Comment 4•18 years ago
|
||
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+
Assignee | ||
Updated•17 years ago
|
Assignee | ||
Comment 5•17 years ago
|
||
Attachment #208578 -
Attachment is obsolete: true
Comment 6•17 years ago
|
||
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.
Description
•