Closed
Bug 1249418
Opened 8 years ago
Closed 8 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•8 years ago
|
||
Minimized with pngcrush (~39 KB vs. ~61 KB).
Reporter | ||
Comment 2•8 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•8 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•8 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•8 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•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/516a92d694dd
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 8•8 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•8 years ago
|
||
yeah - who drinks from a coffee mug on the beach!
Assignee | ||
Comment 10•8 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•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/7c64765b8752
Comment 13•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/2d8efe1a5bd6
Comment 14•8 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•8 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•3 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
•