Closed
Bug 400838
Opened 17 years ago
Closed 17 years ago
The Search: label in Places Organizer is a button, though it should not be
Categories
(Firefox :: Bookmarks & History, defect, P3)
Tracking
()
VERIFIED
FIXED
Firefox 3 beta2
People
(Reporter: Aleksej, Assigned: dao)
References
()
Details
(Keywords: access)
Attachments
(1 file)
2.79 KB,
patch
|
asaf
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9a9pre) Gecko/2007102304 Minefield/3.0a9pre chrome://browser/content/places/places.xul#scopebartitle <toolbarbutton xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" id="scopeBarTitle" label="Search:"/> Steps to Reproduce: 1. Go to the URL, or open Bookmarks → Organize Bookmarks 2. Type something into the search field, so a bar with mode buttons appears. 3. Hover and click the “Search” on its left. Actual results: It’s a button, and is acting like a button. Expected results: It's a label, so nothing should happen. Maybe a little tooltip, though.
Flags: blocking-firefox3?
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Target Milestone: --- → Firefox 3 M10
Assignee | ||
Comment 1•17 years ago
|
||
Updated•17 years ago
|
Whiteboard: [has patch][needs review mano]
Comment 2•17 years ago
|
||
Comment on attachment 287537 [details] [diff] [review] patch r=mano
Attachment #287537 -
Flags: review?(mano) → review+
Updated•17 years ago
|
Keywords: checkin-needed
Whiteboard: [has patch][needs review mano] → [has patch][has review]
Updated•17 years ago
|
Priority: -- → P3
Comment 3•17 years ago
|
||
Checking in browser/components/places/content/advancedSearch.inc; /cvsroot/mozilla/browser/components/places/content/advancedSearch.inc,v <-- advancedSearch.inc new revision: 1.8; previous revision: 1.7 done Checking in browser/components/places/content/places.xul; /cvsroot/mozilla/browser/components/places/content/places.xul,v <-- places.xul new revision: 1.93; previous revision: 1.92 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Summary: The Search: label in Places Organizer is a button, though it should not be. → The Search: label in Places Organizer is a button, though it should not be
Reporter | ||
Comment 4•17 years ago
|
||
Verified fixed with Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9b2pre) Gecko/2007112604 Minefield/3.0b2pre
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 5•17 years ago
|
||
This is unlikely to regress, probably doesn't need a litmus test.
Flags: in-litmus-
Assignee | ||
Updated•15 years ago
|
Whiteboard: [has patch][has review]
Comment 6•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
•