Closed
Bug 603300
Opened 12 years ago
Closed 12 years ago
Add Bing to the default en-GB search plugins; remove Answers.com and CC
Categories
(Mozilla Localizations :: en-GB / English (United Kingdom), defect)
Mozilla Localizations
en-GB / English (United Kingdom)
Tracking
(blocking2.0 beta7+)
RESOLVED
FIXED
mozilla2.0b7
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta7+ |
People
(Reporter: kev, Assigned: mamozrk)
References
Details
(Keywords: productization, Whiteboard: [l10n-src-verification updated])
Attachments
(2 files)
1.24 KB,
patch
|
stas
:
review-
|
Details | Diff | Splinter Review |
1.24 KB,
patch
|
stas
:
review+
|
Details | Diff | Splinter Review |
Add the Bing search plugin to the default list of providers in the search bar. The plugin will be placed in the third position of the Search Bar drop down after Google and Yahoo!. Remove Answers.com and Creative Commons search plugins from the default set included in en-GB. The expected search bar query for builds that incorporate official branding generated by this plugin is: http://www.bing.com/search?q={searchTerms}&form=MOZSBR&pc=MOZI The expected search bar query for builds that do not incorporate official branding generated by this plugin is: http://www.bing.com/search?q={searchTerms}&form=MOZSBR
Attachment #482218 -
Flags: review?(stas)
Reporter | ||
Updated•12 years ago
|
blocking2.0: --- → ?
Updated•12 years ago
|
blocking2.0: ? → beta7+
Comment 1•12 years ago
|
||
Comment on attachment 482218 [details] [diff] [review] Add Bing to search bar in 3rd position, remove Answers and CC from defaults >diff -r 780882cf094d browser/chrome/browser-region/region.properties >+browser.search,order.3=Bing There's a comma here ("search,order"), should be a dot.
Attachment #482218 -
Flags: review?(stas) → review-
Reporter | ||
Comment 2•12 years ago
|
||
The comma was there to give me time to reflect on how awesome I am with this stuff. Corrected.
Attachment #482542 -
Flags: review?(stas)
Updated•12 years ago
|
Attachment #482542 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•12 years ago
|
||
Comment on attachment 482542 [details] [diff] [review] Updated patch to address comment #1 :) Looks good now, thanks! r=me Obviously, this can only land only after bug 603298 lands.
Attachment #482542 -
Flags: review?(stas) → review+
Reporter | ||
Updated•12 years ago
|
Attachment #482542 -
Attachment is patch: true
Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=603298 landed.
Comment 5•12 years ago
|
||
Then this can go on mozilla-central default and the b7 relbranch
Comment 6•12 years ago
|
||
... to remove confusion, this wants to go on l10n-central/en-GB, default. No relbranch over there.
Comment 7•12 years ago
|
||
Oh, right, do eeeet. :)
Assignee | ||
Comment 8•12 years ago
|
||
landed: http://hg.mozilla.org/l10n-central/en-GB/rev/2f515b0b2242
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 9•12 years ago
|
||
Cool, thanks, Mark. Mind sending a sign-off our way? Don't mind the missing strings, those won't block.
Comment 10•12 years ago
|
||
https://hg.mozilla.org/l10n/l10n-src-verification/rev/12cbc5fdd1e2
Whiteboard: [l10n-src-verification updated]
You need to log in
before you can comment on or make changes to this bug.
Description
•