Closed
Bug 1249418
Opened 9 years ago
Closed 9 years ago
Update illustration image of Sync slide
Categories
(Firefox for Android Graveyard :: First Run, defect)
Tracking
(firefox46 verified, firefox47 verified, firefox48 verified)
VERIFIED
FIXED
Firefox 48
People
(Reporter: antlam, Assigned: liuche)
References
Details
Attachments
(3 files)
60.56 KB,
image/png
|
Details | |
38.65 KB,
image/png
|
Details | |
58 bytes,
text/x-review-board-request
|
ahunt
:
review+
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details |
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.
Comment 1•9 years ago
|
||
Minimized with pngcrush (~39 KB vs. ~61 KB).
Reporter | ||
Comment 2•9 years ago
|
||
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)
Assignee | ||
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/40215/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/40215/
Attachment #8730890 -
Flags: review?(ahunt)
Assignee | ||
Comment 4•9 years ago
|
||
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 5•9 years ago
|
||
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+
Comment 7•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 8•9 years ago
|
||
(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)
Reporter | ||
Comment 9•9 years ago
|
||
yeah - who drinks from a coffee mug on the beach!
Assignee | ||
Comment 10•9 years ago
|
||
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+
Comment 12•9 years ago
|
||
bugherder uplift |
Comment 13•9 years ago
|
||
bugherder uplift |
Comment 14•9 years ago
|
||
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)
Updated•9 years ago
|
Comment 15•8 years ago
|
||
Since this has all flags verified I'll mark it as verified fixed.
Status: RESOLVED → VERIFIED
Comment 16•6 years ago
|
||
Based on last updates I will remove the qe-verify flag, thanks.
Flags: qe-verify+
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•