Closed Bug 797039 Opened 8 years ago Closed 8 years ago

Use updated icon for notifications

Categories

(Firefox for Android :: Theme and Visual Design, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 19

People

(Reporter: snorp, Assigned: sriram)

References

Details

Attachments

(1 file)

Design has given us a nice icon to use in status notifications. I've made the updater use it, but it would probably be wise to use it for all the notifications where we currently use R.icon.
The new icon is going in as part of bug 797036, and is named 'ic_status_logo'.
Depends on: 797036
Assignee: nobody → sriram
Attached patch PatchSplinter Review
This changes the logo to be ic_status_logo.

However, I have a doubt: Is ic_status_logo brand specific? (I don't think so). So, do we show a firefox logo for fennec / nightly?
Attachment #677567 - Flags: review?(mark.finkle)
Comment on attachment 677567 [details] [diff] [review]
Patch

We would show the "firefox" notification icon. We don't have channel branded versions yet, and may not.
Attachment #677567 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/131ff198d8c3
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
You need to log in before you can comment on or make changes to this bug.