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)
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•19 years ago
|
Assignee: firefox → nobody
QA Contact: bugzilla → menus
Comment 1•19 years ago
|
||
In which other cases in Firefox do you see this?
Assignee | ||
Comment 2•19 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Attachment #208578 -
Flags: review?(mconnor) → review?(gavin.sharp)
Comment 3•18 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•18 years ago
|
Attachment #208578 -
Flags: ui-review?(beltzner)
Comment 4•17 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•16 years ago
|
Assignee | ||
Comment 5•16 years ago
|
||
Attachment #208578 -
Attachment is obsolete: true
Comment 6•16 years ago
|
||
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.
Description
•