Closed
Bug 423250
Opened 16 years ago
Closed 16 years ago
Library search results don't refresh after clearing search box contents
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: kareem.johnson60+bugzilla, Assigned: mak)
Details
Attachments
(1 file)
1.33 KB,
patch
|
dietrich
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b5pre) Gecko/2008031405 Minefield/3.0b5pre Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9b5pre) Gecko/2008031405 Minefield/3.0b5pre Search functionality bug: When searching for a bookmark using the built-in search box, removing the search query text by clicking the 'x' does not refresh the search results. Only items from the search query are displayed, despite the search box being blank. My current workaround is to refresh the view by changing the selected search folder. Back/forward functionality bug: The forward button never activates, and therefore I cannot browse forward into history. Reproducible: Always Steps to Reproduce: 1. 2. 3. My setup includes the google notebook, foxytunes, passwordmaker pro and ie tab extensions. I will qualify this bug with a test running no add-ons.
Updated•16 years ago
|
Component: Bookmarks → Places
QA Contact: bookmarks → places
Version: unspecified → Trunk
the back/forward func. is being address by bug 408860
Assignee | ||
Comment 3•16 years ago
|
||
confirmed here, clearing search does not restore original results
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: bookmarks manager search, back/forward functionality broken → Library search results don't refresh after clearing search box contents
Assignee | ||
Updated•16 years ago
|
Severity: major → normal
Assignee | ||
Comment 4•16 years ago
|
||
Steps to reproduce. 1. Select a folder in the left pane(for example Bookmarks menu) 2. search for "moz" 3. clear search field (with canc or clicking the clear search button) actual result: you see all bookmarks menu contents as uri, you have to select another folder and come back to see again the original content expected result: you see the correct bookmarks menu contents When the user clears the search field we call onPlaceSelectedURI(false) but the _content.place has not changed, so we don't refresh the view, we should instead since we are expecting different contents. I hope we can take this for the final since it's really a big annoyance against search functionality in the Library
Assignee | ||
Comment 5•16 years ago
|
||
probably not a real blocker, but imo should be wanted
Flags: blocking-firefox3?
Comment 6•16 years ago
|
||
Comment on attachment 317293 [details] [diff] [review] patch r=me, thanks
Attachment #317293 -
Flags: review?(mano) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #317293 -
Flags: approval1.9?
Comment 7•16 years ago
|
||
Comment on attachment 317293 [details] [diff] [review] patch a1.9=beltzner
Attachment #317293 -
Flags: approval1.9? → approval1.9+
Updated•16 years ago
|
Flags: wanted-firefox3+
Flags: blocking-firefox3?
Flags: blocking-firefox3-
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Whiteboard: [has patch][has review][has approval]
Comment 8•16 years ago
|
||
mozilla/browser/components/places/content/places.js 1.159
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [has patch][has review][has approval]
Target Milestone: --- → Firefox 3
Comment 9•16 years ago
|
||
verified fixed using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9pre) Gecko/2008042407 Minefield/3.0pre.
Status: RESOLVED → VERIFIED
Comment 10•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•