Closed Bug 238955 Opened 20 years ago Closed 20 years ago

Can't invoke context menu in about:config until an item is selected

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 191410

People

(Reporter: vdvo, Assigned: vdvo)

Details

Attachments

(2 obsolete files)

In about:config, you can't use click the right mouse button on empty space to
invoke the context menu until an item in the list is selected. Therefore, when
the list is empty, there doesn't seem to be a way to add a new preference.

Steps to reproduce:
1. go to about:config
2. type "qweqwe" in the Filter text field
3. there are now no preferences displayed; try adding a new pref - apparently,
you can't
4. now type "eat_space" in the Filter field
5. there is now exactly one preference displayed, but not selected
6. click the right mouse button in the empty space below the one displayed pref
- nothing happens
7. select the one displayed pref with the left mouse button
8. repeat step 6 - now, the right mouse button works
Attached patch Patch to fix describe behaviour. (obsolete) — Splinter Review
Patch against xpfe/global/resources/content/config.js
Attachment #145728 - Flags: review?(alecf)
Comment on attachment 145728 [details] [diff] [review]
Patch to fix describe behaviour.

why do you touch the elements individually in the first case, and with a for
loop in the second case? Fix this inconsistency at the very least..
Attached patch patch version 2 (obsolete) — Splinter Review
Here's a new patch. No for cycle, added comments.

Also, I noticed a JavaScript warning and error when you press Enter when
nothing is selected. This patch corrects that, too - I hope you don't mind me
not filing a new bug for this one-line addition (plus indentation).
Assignee: prefs → vdvo
Attachment #145728 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Comment on attachment 145822 [details] [diff] [review]
patch version 2

Do we need sr, too, or is Alec's review sufficient?
Attachment #145822 - Flags: review?(alecf)
Attachment #145728 - Flags: review?(alecf)
Comment on attachment 145822 [details] [diff] [review]
patch version 2

I need to sr=, you need to get a front end UI person to r= this.
sr=alecf
Attachment #145822 - Flags: review?(alecf) → superreview+
Attachment #145822 - Flags: review?(neil.parkwaycc.co.uk)
Whoops, this conflicts with a patch I just added for bug 191410 :-(
This will be fixed by bug 191410: "new" in the context menu is enabled even when
there's no selection and you're clicking on an empty space. The fix for that has
full reviews already, just needs checkin.

*** This bug has been marked as a duplicate of 191410 ***
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
Product: Browser → Seamonkey
Attachment #145822 - Attachment is obsolete: true
Attachment #145822 - Flags: review?(neil.parkwaycc.co.uk)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: