1190968 - Update Wikipedia search asset

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

Description

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Wikipedia put in a request that the "w" needs to be updated.

Will attach asset shortly

Comment 1

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: wiki.zip

thanks mike! :D (sebastian nominated you heh)

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

I'm a little skeptical because the new asset seems to be missing many base64 encoded characters over the previous one, but it looks like the changes were made successfully.

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Attachment: New asset

Comment 4

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Attachment: MozReview Request: Bug 1190968 - Update Wikipedia search asset. r=margaret

Bug 1190968 - Update Wikipedia search asset. r=margaret

Comment 5

[:Margaret Leibovic]

(In reply to Michael Comella (:mcomella) from comment #2)
> I'm a little skeptical because the new asset seems to be missing many base64
> encoded characters over the previous one, but it looks like the changes were
> made successfully.

Maybe it's simpler to compress? If it looks alright, works for me! :)

Comment 6

[:Margaret Leibovic]

Comment on attachment 8645243 [details]
MozReview Request: Bug 1190968 - Update Wikipedia search asset. r=margaret

https://reviewboard.mozilla.org/r/15477/#review13979

Ship It!

Comment 7

[Richard Newman [:rnewman]]

Attachment: iOS current state

This is what the iOS search bar currently looks like.

I don't see a difference (if anything, the iOS one looks higher-res and altogether more pleasant).

UXers: if you do, could you file a bug in Firefox for iOS :: Theme & Visual Design to make the same change there?

If not, please just clear your flags and enjoy your Tuesday :)

Comment 8

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

(In reply to Richard Newman [:rnewman] from comment #7)
> I don't see a difference (if anything, the iOS one looks higher-res and
> altogether more pleasant).

fwict, the new asset is slightly shorter (by a few pixels) and the serif at the top of the W's are wider.

Comment 9

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

I'm not sure about the iOS one, so I'm going to leave this to Robin

Comment 10

[Robin Andersen [:tecgirl]]

(In reply to Richard Newman [:rnewman] from comment #7)
> Created attachment 8646147 [details]
> iOS current state
> 
> This is what the iOS search bar currently looks like.
> 
> I don't see a difference (if anything, the iOS one looks higher-res and
> altogether more pleasant).
> 
> UXers: if you do, could you file a bug in Firefox for iOS :: Theme & Visual
> Design to make the same change there?
> 
> If not, please just clear your flags and enjoy your Tuesday :)

The difference is slight, http://c.tecgirl.com/cCtl, and since I made the iOS asset, I can certainly update it to match the negative space and baseline. Filed: bug 1193421

Comment 11

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

url:        https://hg.mozilla.org/integration/fx-team/rev/9b4fa1dc91fd42a5ecc095bcf194ed98f51209d9
changeset:  9b4fa1dc91fd42a5ecc095bcf194ed98f51209d9
user:       Michael Comella <[e-mail]>
date:       Fri Aug 07 16:51:21 2015 -0700
description:
Bug 1190968 - Update Wikipedia search asset. r=margaret

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/9b4fa1dc91fd

Comment 13

[Francesco Lodolo [:flod]]

Comment on attachment 8645243 [details]
MozReview Request: Bug 1190968 - Update Wikipedia search asset. r=margaret

Going ahead and asking for approval on other branches.

Approval Request Comment
[Feature/regressing bug #]: icon currently used for Wikipedia searchplugin doesn't met Wikimedia's branding guidelines
[User impact if declined]: they'll get one icon for 2 cycles (larger icons are in fx41, not fx40), then the correct one in Firefox 43
[Describe test coverage new/current, TreeHerder]: landed in m-c, no code changes involved.
[Risks and why]: -
[String/UUID change made/needed]: -

Comment 14

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Michael, when I was reviewing this patch for uplift I noticed that the xml file that has changed is specific to en-us locale (mobile/locales/en-US/searchplugins/wikipedia.xml). Is the searchplugin image change applicable to other locales that FF ships with? Thanks.

Comment 15

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Redirect comment 14 to antlam.

Comment 16

[Francesco Lodolo [:flod]]

(In reply to Ritu Kothari (:ritu) from comment #14)
> Michael, when I was reviewing this patch for uplift I noticed that the xml
> file that has changed is specific to en-us locale
> (mobile/locales/en-US/searchplugins/wikipedia.xml). Is the searchplugin
> image change applicable to other locales that FF ships with? Thanks.

That is covered by bug 1179109. 

Sadly updating searchplugins for localizations takes a lot longer. I don't plan to uplift up to Beta, but most of the locales are already fixed on both aurora and central.

Comment 17

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

(In reply to Francesco Lodolo [:flod] from comment #16)
> (In reply to Ritu Kothari (:ritu) from comment #14)
> > Michael, when I was reviewing this patch for uplift I noticed that the xml
> > file that has changed is specific to en-us locale
> > (mobile/locales/en-US/searchplugins/wikipedia.xml). Is the searchplugin
> > image change applicable to other locales that FF ships with? Thanks.
> 
> That is covered by bug 1179109. 
> 
> Sadly updating searchplugins for localizations takes a lot longer. I don't
> plan to uplift up to Beta, but most of the locales are already fixed on both
> aurora and central.

Great! Thanks Flod. I will approve for uplift to Beta.

Comment 18

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8645243 [details]
MozReview Request: Bug 1190968 - Update Wikipedia search asset. r=margaret

Approved for uplift to Aurora and Beta. Patch seems safe as it's a image replacement only.

Comment 19

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Requesting QE team to verify the updated icon for wikipedia search. Thanks!

Comment 20

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/b021448f2a38

Comment 21

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/5008ee4e0c1e

Comment 22

[u421692]

Verified as fixed on Firefox 41 Beta 2, on latest Aurora, and latest Nightly.