Closed Bug 967396 Opened 6 years ago Closed 6 years ago

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

Categories

(Mozilla Localizations :: fr / French, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: flod, Assigned: flod)

References

Details

Attachments

(1 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.
Attachment #8375489 - Flags: review?(mconnor)
Attachment #8375489 - Flags: review?(mconnor) → review+
http://hg.mozilla.org/releases/l10n/mozilla-beta/fr/rev/940c63fba06e
http://hg.mozilla.org/releases/l10n/mozilla-aurora/fr/rev/940c63fba06e
http://hg.mozilla.org/l10n-central/fr/rev/940c63fba06e

Théo, can you request a new sign-off on Beta and mark the bug verify after next build?

Please check that everything looks fine on aurora and central, since I've used merge to move the changeset and I see a difference in venkman.properties
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(tchevalier)
Resolution: --- → FIXED
(In reply to Francesco Lodolo [:flod] from comment #2)
> http://hg.mozilla.org/releases/l10n/mozilla-beta/fr/rev/940c63fba06e
> http://hg.mozilla.org/releases/l10n/mozilla-aurora/fr/rev/940c63fba06e
> http://hg.mozilla.org/l10n-central/fr/rev/940c63fba06e
> 
> Théo, can you request a new sign-off on Beta and mark the bug verify after
> next build?

Sign-off requested on fx_beta, thanks for the patch :)

> 
> Please check that everything looks fine on aurora and central, since I've
> used merge to move the changeset and I see a difference in venkman.properties

Venkman fixed, thanks. (A string has been removed for no reason during a previous uplift)
Flags: needinfo?(tchevalier)
Tested with today's Nightly on Win8, working as expected :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.