1208377 - Fennec: Update icon to high-res for bol.com searchplugin

Status: VERIFIED FIXED

(Mozilla Localizations :: nl / Dutch, defect)

Description

[Wim]

Attachment: Logo_bol-com-96px.png

According to https://bugzilla.mozilla.org/show_bug.cgi?id=1179109 I have created a new image for the searchplugin to be used in the Dutch (nl) and Frisian (fy-NL) Firefox and Firefox for Android.

Comment 1

[Francesco Lodolo [:flod]]

Comment on attachment 8665847 [details]
Logo_bol-com-96px.png

Thanks Wim, the image looks great. I'll use this bug as a reference to update both Dutch and Frisian.

Comment 2

[Onno Ekker [:nONoNonO UTC+1]]

The corners of the logo are white, because the original logo is round. Maybe fill the corners with the same blue color or make the logo a but smaller, so that it is round like the original?

Comment 3

[Francesco Lodolo [:flod]]

(In reply to Onno Ekker [:nONoNonO UTC+1] from comment #2)
> The corners of the logo are white, because the original logo is round. Maybe
> fill the corners with the same blue color or make the logo a but smaller, so
> that it is round like the original?

The corners are transparent, I have no clue why Firefox shows them white (I've seen it before). If you download the image and look it offline, you'll see the transparent background.

Having rounded corners is one of the specs to maintain visual consistency with other searchplugins. Take a look here: https://l10n.mozilla-community.org/~flod/p12n/bug1179109_highres_fennec/

Comment 4

[Onno Ekker [:nONoNonO UTC+1]]

(In reply to Francesco Lodolo [:flod] from comment #3)
> The corners are transparent, I have no clue why Firefox shows them white
> (I've seen it before). If you download the image and look it offline, you'll
> see the transparent background.
> 
> Having rounded corners is one of the specs to maintain visual consistency
> with other searchplugins. Take a look here:
> https://l10n.mozilla-community.org/~flod/p12n/bug1179109_highres_fennec/

OK, sorry for the noise.

Comment 5

[Francesco Lodolo [:flod]]

Dutch
https://hg.mozilla.org/releases/l10n/mozilla-beta/nl/rev/bf176c92b4a4
https://hg.mozilla.org/releases/l10n/mozilla-aurora/nl/rev/bf176c92b4a4
https://hg.mozilla.org/l10n-central/nl/rev/bf176c92b4a4

Frisian
https://hg.mozilla.org/releases/l10n/mozilla-beta/fy-NL/rev/e01cd3c58e31
https://hg.mozilla.org/releases/l10n/mozilla-aurora/fy-NL/rev/e01cd3c58e31
https://hg.mozilla.org/l10n-central/fy-NL/rev/e01cd3c58e31

If you could verify that everything works as expected in the next aurora update and mark the bug as verified it would be great. Don't forget to ask a new sign-off on beta.

Comment 6

[Wim]

Tested in latest Mobile Aurora and it looks fine.
Already signed off on beta for fy-NL.
@Ton, can you handle Dutch?