View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003190 | MMW v4 | Playlist / Search | public | 2007-06-26 14:13 | 2007-07-16 21:58 |
| Reporter | Assigned To | ||||
| Priority | immediate | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.0 | ||||
| Fixed in Version | 3.0 | ||||
| Summary | 0003190: Clean up search bar & Persistence | ||||
| Description | the search method is always set to "search whole library"... when MM starts, even when you change it to "search within current node" before close MM. | ||||
| Additional Information | http://www.mediamonkey.com/forum/viewtopic.php?t=19041 | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1050 | ||||
|
|
Note: deleted Dale's other Search-related comments (will enter as new bugs). |
|
|
I've done it this way because I think that this should not be persistent after restart similar like 'All text fields' is always set on the same toolbar. So for the consistency it should not be persistent and "Search whole library" should be always set as default when MM starts. Assigning to Rusty for review, re-assign to me if you don't agree. |
|
|
If you click on a main node (artist, title, etc.) and search using "search current node" you're essentially searching the whole library anyway. (Which means that searching the whole library is 'available' when search current node is set) However, this ability is not available in reverse (ie, if "search whole library" is on, there is no way to search just the current node). Therefore, alot of people will want to make "search current node" the default as they will never want to use "search in whole library". Also, don't forget that a user can always hit ctrl-f (or choose it in the drop down) to search the whole library when needed... this isn't available in reverse either. |
|
|
I tend to agree with Dale that search based on node is going to be the preferred approach for most users. I also think that the setting for Node vs Entire library is very different from the settings (All text, Artist, Album, Title), since it is a configuration change that the user will want to persist. Side note: From my perspective I think that context-dependent search should probably be the only search type since it: -allows for all types of searches (as dale pointed out) -is Needed for functions such as searching the OPML directory for a particular podcast -would simplify the search bar a fair bit since we can: -- get rid of options like Amazon, Barnes & Noble, etc. (those would be searchable if the user selected the relevant node) -- move the search options of (Search whole library, Search current node only) to the right side -- get rid of the 'Search:' text to the left of the search box and instead place it directly within the text box. Comments appreciated, though we'll probably have to track this in another bug. |
|
|
Following discussion with Jiri, I'm listing all of the changes within this bug since persistence is closely related with other design issues: The current search bar is a bit too complicated, giving users too many options (i.e. considering that MM offers context-sensitive searching, there isn't a need to offer so many extra options), and splitting them across several locations. Suggested changes: 1) Change context-sensitive search to the default behaviour (confirm via forum) 2) get rid of the 'Search:' text to the left of the search box and instead place 'search' directly within the text box. 3) Move the search options from the 'search:' dropdown to the right side of the search button, and reduce available options. i.e. present the following options: Currently selected node Library (all text fields) ------------------------- Amazon Barnes&Noble CD Universe All Posters Persistence would be to whichever of the first 2 settings was last used. |
|
|
All is resolved in build 1047 except 2) get rid of the 'Search:' text to the left of the search box and instead place 'search' directly within the text box. How do you mean this, do you mean that the 'search' text should be overrwritten by text to search once user types the text or the 'search' text should persist and text to search should be placed next to the 'search' text? |
|
|
Yes, this is exactly what I mean. For an example of this see Firefox. When the search box is empty, it contains greyed out text such as 'Search Google'. This text disappears as soon as the user starts typing anything in the text box. |
|
|
Based on feedback at: http://www.mediamonkey.com/forum/viewtopic.php?t=19041, we might want to modify two things: 4) Currently selected node Entire Library Library (filtered) ------------------------- Amazon .... 5) Instead of just showing 'Search' in the textbox, show whatever the user has selected as the search destination. See the forum post if this isn't clear. |
|
|
Might as well make it in order of most likely used. Currently selected node Library (filtered) Entire Library ------------------------- Amazon .... |
|
|
2) and 5) fixed in build 1049. |
|
|
So the remaining open issue is: 4) Currently selected node Entire Library Library (filtered) ------------------------- Amazon .... Note that I suggest to list it in this order (rather than Dale's revised order) since this is the order used for these filters every in MM (i.e. Entire Library prior to filters). --------------- I've raised the bug to Immediate, though, because the implementation in build 1049 is very difficult to use because of several issues that need to be resolved before Alpha6: 6) When I click the search bar, the contents of the last search appear, but without the first character. This is very annoying since the user expects the field to initially be blank (instead of starting off with a meaningless search constraint). 7) The text that appears in the text box is too long and is truncated. It should be changed to: -Current selection -Entire Library 8) If the user deletes the text in the search bar, then the search constraints should be loosened (i.e. all the contents of the node should appear again). |
|
|
And I would say: 9) Remove "Barnes Noble" because it always show message: "This merchant is no longer participating in the affiliate program. " |
|
|
All items are fixed in build 1050. |
|
|
verified 1050. Nice work. |
