Closed Bug 1152477 Opened 5 years ago Closed 5 years ago

Update Theme Editor Home Screen Icon

Categories

(Firefox OS Graveyard :: Gaia::Theme Editor, defect, P1)

x86
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: epang, Assigned: drs)

References

Details

(Whiteboard: [spark])

Attachments

(2 files, 1 obsolete file)

37 bytes, text/x-github-pull-request
etienne
: review+
Details | Review
139.38 KB, application/x-zip-compressed
Details
Attached file Theme_Editor_Homescreen_Icons.zip (obsolete) —
Attached is the updated theme editor homescreen icon.
Hey guys, can whoever has a spare moment help update the icon?  Thanks!
Flags: needinfo?(kgrandon)
Flags: needinfo?(etienne)
Flags: needinfo?(drs)
Attached file Update app icons.
Whoever reviews this first, please go ahead and merge.
Assignee: nobody → drs
Status: NEW → ASSIGNED
Flags: needinfo?(drs)
Attachment #8589838 - Flags: review?(kgrandon)
Attachment #8589838 - Flags: review?(etienne)
Eric, I don't think a 512x512 icon is really necessary. Could you provide one at 256x256, instead? This is closer to the dimensions that we'll expect on the Aries device.
Flags: needinfo?(epang)
Flags: needinfo?(etienne)
Attachment #8589838 - Flags: review?(kgrandon)
Attachment #8589838 - Flags: review?(etienne)
Attachment #8589838 - Flags: review+
Hope I didn't merged too eagerly :)

https://github.com/fxos/studio/pull/4
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Thanks, it's ok. I'll just merge another change without review when we get the 256x256 icon.
Flags: needinfo?(kgrandon)
Blocks: spark-theme-editor
No longer blocks: spark-customizer
Priority: -- → P1
Updated set of icons including the 256x256px
Attachment #8589830 - Attachment is obsolete: true
Flags: needinfo?(epang) → needinfo?(drs)
Whiteboard: [lightsaber] → [ignite]
Whiteboard: [ignite] → [spark]
Component: Gaia → Gaia::Theme Editor
You created a remote branch 1152477-icons in the "fxos/studio" repository, I don't have the right to delete it but likely you have.
Flags: needinfo?(drs)
Deleted, thanks.
Flags: needinfo?(drs)
You need to log in before you can comment on or make changes to this bug.