Rename and move "Search Options" menu item

VERIFIED FIXED in M18

Status

P2
normal
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: johng, Assigned: bugzilla)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta2+] [Fix in hand] EXCEPTION FEATURE)

Attachments

(3 attachments)

(Reporter)

Description

19 years ago
The Search menu currently includes a "Search Options" item, but the text needs 
to be changed and the placement within the menu needs to be changed.

*Naming*

Rename the menu item from "Search Options" to:
My Sidebar Search Tab

And rename the sub-menu options from "simple" and "advanced" to:
Basic
Advanced

*Placement within menu*

In general, it should be moved to the bottom or almost the bottom of the menu 
and be set apart with a separator line.

More specifically, a separator line should be placed just above this menu item. 
 In the mozilla build, this menu item should be the very last item in the Search 
menu.  In the Netscape branded build, it should be the second to last item, just 
before the "Internet Keywords" item (no separator line between these two, just 
one above the first one).

*Additional notes*

When you pull "Search Options" out of the higher location in the search menu 
where it is now, you should also remove the separator line that comes with it.  
The end result is to have no separator lines between the following search menu 
items: "Search the Web", "Search Bookmarks/History", "Search Mail", "Search 
Address Book".

Also note that the "Internet Keywords" item will probably be renamed to "Using 
Search and Keywords" and will link to online help.  Verah is assigning a 
separate bug for this item.
(Reporter)

Comment 1

19 years ago
Because of the usability concerns we have with this menu item (which was our 
compromise in a mtg with Jim Martin), we want to get feedback during PR2.  It is 
also a very easy, low risk fix.

Therefore nominating for nsbeta2 and marking M17.
Keywords: nsbeta2
Target Milestone: --- → M17

Comment 2

19 years ago
Yep, I think we need to do this.  Ben, how hard is it to do all this?

Assignee: don → ben
Priority: P3 → P2
Target Milestone: M17 → M18

Comment 3

19 years ago
[nsbeta2+] exception feature part of compromise with Netcenter.
Whiteboard: [nsbeta2+] EXCEPTION FEATURE
(Assignee)

Comment 4

19 years ago
I can do this right now
Assignee: ben → BlakeR1234
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 5

19 years ago
Created attachment 9941 [details] [diff] [review]
patch (for mozilla)
(Assignee)

Comment 6

19 years ago
Created attachment 9942 [details] [diff] [review]
patch part 2
(Assignee)

Comment 7

19 years ago
Oops..  Sorry, just noticed two [minor] problems with my patches:

* the first patch for navigatorOverlay.xul also contains the tiny patch for 
bug 37537, which had been in my tree at the time.

* the second patch for navigator.dtd changes the build ID to 2000060908

Once these are reviewed, I'll resubmit the patches if you wish.  I guess the 
patch for 37537 can just be reviewed and checked in along with this one, but 
I'll fix the build ID one after review.
Keywords: patch
(Assignee)

Comment 8

19 years ago
sigh. sorry folks, I also just noticed that the patch moves "Search the Web" 
below "Search Addressbook", instead of leaving it above "Search 
Bookmarks/History" as it should have been.  I'm _really_ not this dumb, this is 
just further proof that 3:45 AM is not the time to be making patches.

I'll attach some much better patches tomorrow...for now, let me know if these 
satisfy the general idea of what you wanted ('xcept for what i just mentioned)
(Assignee)

Comment 9

19 years ago
Created attachment 9961 [details] [diff] [review]
good navigatorOverlay.xul patch (for mozilla) - ignore the old "patch (for mozilla)"
(Assignee)

Comment 10

19 years ago
sorry about all the spam from this, everyone.  patches are now fine...ignore 
the "patch (for mozilla)" and use the second and third patches.

john - you didn't specify which accelerator key you want "My Sidebar Search 
Tab" to use, however.
(Assignee)

Updated

19 years ago
Whiteboard: [nsbeta2+] EXCEPTION FEATURE → [nsbeta2+] [Fix in hand] EXCEPTION FEATURE
(Assignee)

Comment 11

19 years ago
fixed checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

19 years ago
ben changed this to "My Sidebar Search Panel" (rather than "tab"), presumably 
for a good reason...other than that, though, this looks fixed to me..

Comment 13

19 years ago
It should be "My Sidebar Search Tab" -- these thingys are tabs, not panels. 
John, do you want to reopen for this? Or should we call it "UI text polish" and 
fix it next time?
(Assignee)

Comment 14

19 years ago
I agree that it should be tab...that's how they're referred to throughout the 
rest of the app. ben, why'd you change it?
(Assignee)

Comment 15

19 years ago
Ben wrote and explained why he made the change to "panels" instead of "tabs", 
and his reasoning makes a lot of sense -- the different components of the 
sidebar might not resemble tabs at all in other skins, despite their appearance 
in the modern skin.  He also explained that there are better things in store 
for the sidebar after the 6.0 release, and we shouldn't limit ourselves to 
calling them tabs because of this ("panels" is much more general).

in any case, can we verify this please?

Comment 16

19 years ago
johng I'd ask that you take a quick glance at this to be sure but based on your original writeup and what I
can verify in the builds I'd say this was fixed. The only thing I'm not 100% on is the position of the
My Sidebar Search Panel menuitem within the menu and the use of separators around it.

I'm presuming these are correct and marking VERIFIED(2000062308). If they are not, feel free to reopen.
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.