Closed
Bug 603300
Opened 15 years ago
Closed 15 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•15 years ago
|
blocking2.0: --- → ?
Updated•15 years ago
|
blocking2.0: ? → beta7+
Comment 1•15 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•15 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•15 years ago
|
Attachment #482542 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•15 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•15 years ago
|
Attachment #482542 -
Attachment is patch: true
Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=603298 landed.
Comment 5•15 years ago
|
||
Then this can go on mozilla-central default and the b7 relbranch
Comment 6•15 years ago
|
||
... to remove confusion, this wants to go on l10n-central/en-GB, default. No relbranch over there.
Comment 7•15 years ago
|
||
Oh, right, do eeeet. :)
| Assignee | ||
Comment 8•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 9•15 years ago
|
||
Cool, thanks, Mark.
Mind sending a sign-off our way? Don't mind the missing strings, those won't block.
Comment 10•14 years ago
|
||
Whiteboard: [l10n-src-verification updated]
You need to log in
before you can comment on or make changes to this bug.
Description
•