Update Yahoo Search Plugin (sync with Firefox) [20161231]

RESOLVED FIXED in seamonkey2.50

Status

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: philip.chee, Assigned: philip.chee)

Tracking

SeaMonkey 2.50 Branch
seamonkey2.50

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Only change is updating the Yahoo! logo to the current one.
Posted patch Patch v1.0Splinter Review
Attachment #8824116 - Flags: review?(iann_bugzilla)
Posted image Yahoo Logo (obsolete) —
Attachment #8824119 - Attachment mime type: text/plain → image/x-icon
Attachment #8824119 - Attachment filename: file_1328920.txt → yahoo.ico
Attachment #8824119 - Attachment is obsolete: true
Attachment #8824122 - Attachment mime type: image/x-ico → image/x-icon
Comment on attachment 8824116 [details] [diff] [review]
Patch v1.0

r/a=me
Do we need to look at centralising searchplugins too?
Attachment #8824116 - Flags: review?(iann_bugzilla) → review+
Also compare the merging of the two <Url type="test/html"> entries in the Firefox version or should that be another bug to cover all affected searchplugins?
Flags: needinfo?(philip.chee)
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/13d07cf86fa66c3068c84d056c9e5bc3bc0f6ce4
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.50
(In reply to Ian Neal from comment #4)
> Comment on attachment 8824116 [details] [diff] [review]

> Do we need to look at centralising searchplugins too?
Not sure if we can since unlike Firefox, our translators traditionally have autonomy on which search plugins to include in their locales.

(In reply to Ian Neal from comment #5)
> Also compare the merging of the two <Url type="test/html"> entries in the
> Firefox version or should that be another bug to cover all affected
> searchplugins?
Will do this in a separate bug.
Flags: needinfo?(philip.chee)
You need to log in before you can comment on or make changes to this bug.