823472 - [FTU UX VD] Replace "Hello" bubbles illustration asset in lenguage selection

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Description

[Victoria Gerchinhoren [:vicky]]

Attachment: New asste for lenguage selection screen

Please replace that asset in lenguage selection. THe new asset replaces the OPen Sans font, and uses the MozTT one. Find attached the file.

Comment 1

[Sam Joch [:samjoch]]

Landed in uxbranch: https://github.com/gordonbrander/gaia/commit/52844c866907a35be646cfc03f2c4055c841b899

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

qa-verified

Comment 3

[Maria Angeles Oteo (:oteo)]

Hi Fernando or Sam, after the Pull Request corresponding revision can you land the patch in master? 

As there is no clear answers about the UX branch purpose right now, it'd be better to land it in master so after that, we can ask for uplifting the patch to v1-train. Otherwise I am afraid the work done will be lost and it will not land in time or never in the commercial release v1.1.

Comment 4

[Fernando Campo (:fcampo)]

We have a conflict on this one, as the current patch modifies an image that no longer exists (was changed time ago), so I'm waiting for Victoria to finish her PTO so she can guide us on here(In reply to Maria 

Angeles Oteo:oteo from comment #3)
> Hi Fernando or Sam, after the Pull Request corresponding revision can you
> land the patch in master?

Comment 5

[Victoria Gerchinhoren [:vicky]]

Attachment: The correct image, to replace

Comment 6

[Fernando Campo (:fcampo)]

Attachment: Link to https://github.com/mozilla-b2g/gaia/pull/8940

New image added, PR to master

Comment 7

[Sam Joch [:samjoch]]

Comment on attachment 732754 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/8940

Master commit is https://github.com/mozilla-b2g/gaia/commit/40608b6ea83347c4a0dfc4e4d1f4571d99b88d9f


NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): -
User impact if declined:  -
Testing completed: Yes
Risk to taking this patch (and alternatives if risky): low risk, mostly an image updates
String or UUID changes made by this patch: -

Comment 8

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 732754 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/8940

This comes from the UX branch process where QA has already checked that it works. a=me.

Comment 9

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Master build : 2013-04-05-03-05-43
name="mozilla-central" revision="55f9e3e3dae7"  
name="integration/gaia-central" revision="2005e026bdae"
"gecko.git" revision="66b64f47a4eac1bd4f5ea6748b42ddf732297550"
"gaia.git" revision="15468abcba0dd2549f0c102df887a5c049b9c641"
Unagi

Comment 10

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted commit 40608b6ea83347c4a0dfc4e4d1f4571d99b88d9f as:
v1-train: 6438e00da8064c5ad3b61fef52bb6b1c7e6d59d4