Update illustration image of Sync slide

VERIFIED FIXED in Firefox 46

Status

()

VERIFIED FIXED
3 years ago
3 months ago

People

(Reporter: antlam, Assigned: liuche)

Tracking

unspecified
Firefox 48
All
Android
Points:
---

Firefox Tracking Flags

(firefox46 verified, firefox47 verified, firefox48 verified)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
Created attachment 8720983 [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.
Created attachment 8730327 [details]
anywhere_illust2_XXHDPI_PNGCRUSH.png

Minimized with pngcrush (~39 KB vs. ~61 KB).
(Reporter)

Comment 2

3 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

3 years ago
Created attachment 8730890 [details]
MozReview Request: Bug 1249418 - Update illustration image of Sync slide. r=ahunt

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

3 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

3 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

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/516a92d694dd
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox48: --- → fixed
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)
(Reporter)

Comment 9

3 years ago
yeah - who drinks from a coffee mug on the beach!
(Assignee)

Comment 10

3 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?

Updated

3 years ago
status-firefox46: --- → affected
status-firefox47: --- → affected
Flags: qe-verify+
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

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/7c64765b8752
status-firefox47: affected → fixed

Comment 13

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/2d8efe1a5bd6
status-firefox46: affected → fixed

Comment 14

3 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

3 years ago
status-firefox46: fixed → verified
status-firefox47: fixed → verified
status-firefox48: fixed → verified

Comment 15

2 years ago
Since this has all flags verified I'll mark it as verified fixed.
Status: RESOLVED → VERIFIED

Comment 16

3 months ago
Based on last updates I will remove the qe-verify flag, thanks.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.