Closed
Bug 114972
Opened 23 years ago
Closed 23 years ago
Search context menu item should be visible but disabled even when text not selected
Categories
(SeaMonkey :: Search, defect, P3)
SeaMonkey
Search
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: samir_bugzilla, Assigned: samir_bugzilla)
References
Details
(Whiteboard: [adt3])
The search context menu item is currently only enabled when text is selected.
The feature is not visible and as such should be available but disabled with
hintful text such as:
| ___ |
| |d| Search dmoz.org for "..." |
|==================================|
Looking for input from Todd and Marlon.
Assignee | ||
Updated•23 years ago
|
Priority: -- → P3
Target Milestone: --- → mozilla0.9.8
Assignee | ||
Comment 1•23 years ago
|
||
Dan pointed the lack of visiblity of the feature (and this remedy reported
above) when he was reviewing some related code. He is a user who has wanted the
search context menu item but didn't even know it was in the product... and he's
a mozilla developer!
Comment 2•23 years ago
|
||
Right -- I don't often randomly click on a website before opening the context
menu to see if new items have shown up. The presence of disabled items like
"Copy" has trained me not to expect the menu to change just because something is
selected.
Comment 3•23 years ago
|
||
The same thing happened to me too. I didn't know it was already there because I
never saw the item. I think this is the right solution for increasing
discoverablity of the feature even though it does add an additional context menu
item.
Assignee | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9.8 → mozilla0.9.9
Assignee | ||
Comment 4•23 years ago
|
||
Search triage team: nsbeta1+
Marlon, have we accounted for this in the new context menu spec?
Keywords: nsbeta1+
Comment 6•23 years ago
|
||
Marlon needs to make a decision and update the context menu spec before this
gets done, or I'm just going to remove it again with my patch...
Assignee | ||
Updated•23 years ago
|
Whiteboard: [adt3]
Assignee | ||
Comment 7•23 years ago
|
||
Marlon,
Could you indicate if this is going to be part of the final context menu spec?
I already have a patch but would be perfectly happy with leaving it out. Need a
decision soon since drivers and ADT are cracking down on what gets checked in.
Assignee | ||
Comment 8•23 years ago
|
||
Got checked in with bug 106127.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Product: Core → SeaMonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•