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)

x86
All
defect

Tracking

()

VERIFIED FIXED
Firefox 3 beta2

People

(Reporter: Aleksej, Assigned: dao)

References

()

Details

(Keywords: access)

Attachments

(1 file)

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?
Flags: blocking-firefox3? → blocking-firefox3+
Target Milestone: --- → Firefox 3 M10
Attached patch patchSplinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #287537 - Flags: review?(mano)
Whiteboard: [has patch][needs review mano]
Comment on attachment 287537 [details] [diff] [review]
patch

r=mano
Attachment #287537 - Flags: review?(mano) → review+
Keywords: checkin-needed
Whiteboard: [has patch][needs review mano] → [has patch][has review]
Priority: -- → P3
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
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
This is unlikely to regress, probably doesn't need a litmus test.
Flags: in-litmus-
Whiteboard: [has patch][has review]
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.

Attachment

General

Created:
Updated:
Size: