Closed
Bug 1249418
Opened 10 years ago
Closed 10 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 |
anywhere_illust2_XXHDPI.png
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•10 years ago
|
||
Minimized with pngcrush (~39 KB vs. ~61 KB).
|
Reporter | |
Comment 2•10 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•10 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•10 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•10 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•10 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
|
|
||
Comment 8•10 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•10 years ago
|
||
yeah - who drinks from a coffee mug on the beach!
|
|
Assignee | |
Comment 10•10 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 11•10 years ago
|
||
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•10 years ago
|
||
| bugherder uplift | ||
|
|
||
Comment 13•10 years ago
|
||
| bugherder uplift | ||
|
||
Comment 14•10 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•10 years ago
|
|
|
||
Comment 15•9 years ago
|
||
Since this has all flags verified I'll mark it as verified fixed.
Status: RESOLVED → VERIFIED
|
||
Comment 16•7 years ago
|
||
Based on last updates I will remove the qe-verify flag, thanks.
Flags: qe-verify+
|
||
Updated•5 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
•