Closed Bug 1249418 Opened 6 years ago Closed 6 years ago

Update illustration image of Sync slide

Categories

(Firefox for Android Graveyard :: First Run, defect)

All
Android
defect
Not set
normal

Tracking

(firefox46 verified, firefox47 verified, firefox48 verified)

VERIFIED FIXED
Firefox 48
Tracking Status
firefox46 --- verified
firefox47 --- verified
firefox48 --- verified

People

(Reporter: antlam, Assigned: liuche)

References

Details

Attachments

(3 files)

I'd like to update this image to replace the lemonade drink with a coconut. It's obviously no rush but the next round of updates we do should include this.
Minimized with pngcrush (~39 KB vs. ~61 KB).
This is mostly a UX nit, should we get this uplifted as well with bug 1256400? or will that be too much?
Flags: needinfo?(liuche)
Flags: needinfo?(bbermes)
I also crushed one other uncompressed image in the drawables folder down to about a third of the size.
Assignee: nobody → liuche
Flags: needinfo?(liuche)
Comment on attachment 8730890 [details]
MozReview Request: Bug 1249418 - Update illustration image of Sync slide. r=ahunt

https://reviewboard.mozilla.org/r/40215/#review36767

(We seem to be saving 60KB in the first image.)
Attachment #8730890 - Flags: review?(ahunt) → review+
https://hg.mozilla.org/mozilla-central/rev/516a92d694dd
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
(In reply to Anthony Lam (:antlam) from comment #2)
> This is mostly a UX nit, should we get this uplifted as well with bug
> 1256400? or will that be too much?

Just curious, how come? :) Do you think a coconut fits better in this picture?
Flags: needinfo?(bbermes)
yeah - who drinks from a coffee mug on the beach!
Comment on attachment 8730890 [details]
MozReview Request: Bug 1249418 - Update illustration image of Sync slide. r=ahunt

Approval Request Comment
[Feature/regressing bug #]: Update new images for firstrun release
[User impact if declined]: Suboptimal image
[Describe test coverage new/current, TreeHerder]: local testing
[Risks and why]: low, resource change
[String/UUID change made/needed]: none
Attachment #8730890 - Flags: approval-mozilla-beta?
Attachment #8730890 - Flags: approval-mozilla-aurora?
Comment on attachment 8730890 [details]
MozReview Request: Bug 1249418 - Update illustration image of Sync slide. r=ahunt

Low risk, Aurora47+, Beta46+
Attachment #8730890 - Flags: approval-mozilla-beta?
Attachment #8730890 - Flags: approval-mozilla-beta+
Attachment #8730890 - Flags: approval-mozilla-aurora?
Attachment #8730890 - Flags: approval-mozilla-aurora+
Tested using:
Device: Huawei Honor (Android 5.1.1)
Build: Firefox for Android Beta - 47.0b6
Aurora - 48.0a2(2016-05-17)
Nightly - 49.0a1 (2016-05-17)
Since this has all flags verified I'll mark it as verified fixed.
Status: RESOLVED → VERIFIED
Based on last updates I will remove the qe-verify flag, thanks.
Flags: qe-verify+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.