Non-localizable label in Places popup: "Search:"

RESOLVED FIXED in Firefox 2 alpha2

Status

()

Firefox
Bookmarks & History
P1
normal
RESOLVED FIXED
12 years ago
9 years ago

People

(Reporter: u60234, Assigned: Joey Minta)

Tracking

({fixed1.8.1, l12y})

Trunk
Firefox 2 alpha2
fixed1.8.1, l12y
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.1; sv-SE; rv:1.9a1) Gecko/20060413 Firefox/3.0a1

The label for the Places popup search box is hardcoded at
http://lxr.mozilla.org/mozilla/source/browser/components/places/content/placesPopup.xul#65

> <label value="Search:"/>
Priority: -- → P1
Target Milestone: --- → Firefox 2 alpha2
(Assignee)

Comment 1

12 years ago
Created attachment 220358 [details] [diff] [review]
add dtd and use entity

Patch adds the places.dtd to the xul file so that we can use the search.label entity.
Assignee: annie.sullivan → jminta
Status: NEW → ASSIGNED
Attachment #220358 - Flags: review?(annie.sullivan)

Updated

12 years ago
Attachment #220358 - Flags: review?(annie.sullivan) → review+
(Assignee)

Comment 2

12 years ago
Comment on attachment 220358 [details] [diff] [review]
add dtd and use entity

I'm unfamiliar with the Places development flow.  What, if any, additional reviews/approvals do I need here prior to checking in?  Do Places patches still go to both the 1.8 branch and trunk or just trunk?
(Assignee)

Comment 3

12 years ago
Comment on attachment 220358 [details] [diff] [review]
add dtd and use entity

Heh, guess this bug has been sitting here a bit.  mconnor, I assume even though we're not building places on 1_8, we still want to keep them in sync?
Attachment #220358 - Flags: approval-branch-1.8.1?(mconnor)

Comment 4

12 years ago
Branch and trunk for places are currently a bit out of sync. Feel free to check into branch if it's convenient, but don't worry too much about it.
(Assignee)

Comment 5

12 years ago
Patch landed on branch and trunk.
Checking in browser/components/places/content/placesPopup.xul;
/cvsroot/mozilla/browser/components/places/content/placesPopup.xul,v  <--  placesPopup.xul
new revision: 1.1.2.3; previous revision: 1.1.2.2

Checking in browser/components/places/content/placesPopup.xul;
/cvsroot/mozilla/browser/components/places/content/placesPopup.xul,v  <--  placesPopup.xul
new revision: 1.2; previous revision: 1.1
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Comment on attachment 220358 [details] [diff] [review]
add dtd and use entity

not that it matters!
Attachment #220358 - Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
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.