469437 - Searches in bookmarks sub-folders of the Library window should default to the context of All Bookmarks ("Search Bookmarks")

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, defect, P4)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

Currently the search behavior of the Library window is a little strange:

History: Search History
Tags: Search Bookmarks
All Bookmarks: Search Bookmarks
   Bookmarks Toolbar: Search in 'Bookmarks Toolbar'
       Sub-folders; Search Bookmarks
   Bookmarks Menu: Search in 'Bookmarks Menu'
       Sub-folders; Search Bookmarks
   Unfiled Bookmarks: Search in 'Unfiled Bookmarks'
       Sub-folders; Search Bookmarks

For all of the various bookmark folders (various roots and all subfolders) we should default to the widest search possible, "Search Bookmarks."  The parent folder for each search should always be an available context, but never the default context.

Also note bug 469436 for some minor string changes to the context bar.

Comment 1

[Alex Faaborg [:faaborg] (Firefox UX)]

Sorry about all the edits to the summary, not sure on the correct summary.  I think it makes sense to use the phrase "Bookmarks" in the search field instead of "All Bookmarks", so: "Search Bookmarks"

Comment 2

[Marco Bonardo [:mak]]

while working on 3.0 IIRC the request was exactly to make "search in <folder>" the default context when available...

Comment 3

[Alex Faaborg [:faaborg] (Firefox UX)]

>while working on 3.0 IIRC the request was exactly to make "search in <folder>"
>the default context when available...

yeah, I honestly can't remember what I decided was a good default back then, but defaulting to the folder doesn't seem as useful as a more global bookmark search, since folders don't usually contain so many bookmarks that you would need to do a search.

Comment 4

[Drew Willcoxon :adw]

Attachment: v1

This bug raises a couple of issues:

1. If we're defaulting to searching all bookmarks, the label of the "Find in ..." command needs to be updated, too.  It was being updated to "Find in <specific folder>...".  So this patch borrows the OrganizerQueryAllBookmarks property, which in en-us is "All Bookmarks", and the label becomes "Find in All Bookmarks".  At this point might it be better to sync the label with the emptytext of the search box, which reads "Search Bookmarks"?  And now there are only two possibilities for the label: "Find in All Bookmarks" or "Find in History".  Is the "Find in %S ..." property still needed?

2. Until bug 475390 is fixed, it's now impossible to search in bookmarks toolbar, bookmarks menu, and unsorted bookmarks.

Comment 5

[Marco Bonardo [:mak]]

i'll look at the patch soon, for now commenting on the issues:
> 1. If we're defaulting to searching all bookmarks, the label of the "Find in
> ..." command needs to be updated, too.  It was being updated to "Find in
> <specific folder>...".  So this patch borrows the OrganizerQueryAllBookmarks
> property, which in en-us is "All Bookmarks", and the label becomes "Find in All Bookmarks". At this point might it be better to sync the label with the
> emptytext of the search box, which reads "Search Bookmarks"?  And now there are only two possibilities for the label: "Find in All Bookmarks" or "Find in
> History".  Is the "Find in %S ..." property still needed?

comment 0 and 1 are quite clear on this, it should always read "Search Bookmarks" or "Search History", unless you are on one of the roots, in such a case should be "Search in <root>" (so Search in %s is still useful).
I would not use OrganizerQueryAllBookmarks, because All Bookmarks is the name of the folder in the left pane, i find it confusing for the user. Also because we have some issue transating the left pane folder when locale change (We don't do it indeed). Using emptytext is fine i think, the purpose is the same.

I've not checked all the patch, but rememeber to sync also the filter buttons (scopeBar) that appear when you start the search, so All Bookmarks (at this point should be simply Bookmarks, if possible) is selected correctly instead of <folder>

So 2 things should be updated and in sync, the Search field, and the Scope bar.

> 2. Until bug 475390 is fixed, it's now impossible to search in bookmarks
> toolbar, bookmarks menu, and unsorted bookmarks.

you can fix it here as well if you need it and the code touched is the same, i've seen you've taken it.

Comment 6

[Marco Bonardo [:mak]]

and notice in the scope bar we will still have the button "Search: <foldername>", the only difference is that now it is no more the default option.

Comment 7

[Drew Willcoxon :adw]

Related: bug 434704, bug 451151