Search context menu item should be visible but disabled even when text not selected

VERIFIED FIXED in mozilla1.0

Status

P3
normal
VERIFIED FIXED
17 years ago
10 years ago

People

(Reporter: samir_bugzilla, Assigned: samir_bugzilla)

Tracking

Trunk
mozilla1.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt3])

(Assignee)

Description

17 years ago
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

17 years ago
Priority: -- → P3
Target Milestone: --- → mozilla0.9.8
(Assignee)

Comment 1

17 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!  
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

17 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

17 years ago
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Comment 4

17 years ago
Search triage team: nsbeta1+

Marlon, have we accounted for this in the new context menu spec?
Keywords: nsbeta1+
(Assignee)

Updated

17 years ago
Depends on: 106127
Target Milestone: mozilla0.9.9 → mozilla1.0
(Assignee)

Comment 5

17 years ago
Patch is part of fix for bug 106127.
Keywords: patch

Comment 6

17 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

17 years ago
Whiteboard: [adt3]
(Assignee)

Comment 7

17 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.

Updated

17 years ago
Depends on: 75338
(Assignee)

Comment 8

17 years ago
Got checked in with bug 106127.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Updated

17 years ago
No longer depends on: 75338

Comment 9

17 years ago
VERIFIED Fixed with 200020322 builds
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.