Closed Bug 967398 Opened 6 years ago Closed 6 years ago

[it] Update Yahoo searchplugin icon to hi-dpi version for desktop, create Yahoo Metro version

Categories

(Mozilla Localizations :: it / Italian, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: flod, Assigned: flod)

References

Details

Attachments

(1 file, 1 obsolete file)

Hi,
we need to update the current icon used in Yahoo searchplugin to a new hi-dpi version.

Since Firefox Metro is targeting Firefox 28 for release, we also want to switch to a Metro version of Yahoo for your locale.

I'm going to provide directly a patch in this bug, get it reviewed, and land the fix in your mozilla-beta repository. Then try to merge it down (or transplant if merge fails) to mozilla-aurora and l10n-central. 

To be clear: no action is required on your part. This bug is filed to keep track of this change, and so that you're aware of it, since it will require a new sign-off on mozilla-beta to get into next releae.

Please ask if you need any more information.
Attached patch bug967398.patch (obsolete) — Splinter Review
Mike, can you take a look at this patch? I'm using "yahoo-fr-metrofx" instead of "yahoo-fr", but I haven't got any confirmation if that's right.

If that's OK, I'll get the other patches ready for your review and start landing directly on aurora for other locales.

For reference, this is the script I'm currently using
https://github.com/flodolo/scripts/blob/master/mozilla_l10n/searchplugins/fix_yahoo.py
Attachment #8373911 - Flags: review?(mconnor)
Update patch based on bug 942024 ("yahoo-fr-metro").
Attachment #8373911 - Attachment is obsolete: true
Attachment #8373911 - Flags: review?(mconnor)
Attachment #8374657 - Flags: review?(mconnor)
Attachment #8374657 - Flags: review?(mconnor) → review+
Thanks. I'll wait for this to land in a build and then start with the other 4 bugs

http://hg.mozilla.org/releases/l10n/mozilla-beta/it/rev/c8b66f3cd8b7
http://hg.mozilla.org/releases/l10n/mozilla-aurora/it/rev/c8b66f3cd8b7
http://hg.mozilla.org/l10n-central/it/rev/c8b66f3cd8b7
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Verified on Nightly 30.0a1 (2014-02-13), I'll wait for a real beta to mark this as verified (just requested a new sign-off).
Verified on last beta too.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.