Closed
Bug 1279137
Opened 7 years ago
Closed 7 years ago
Update notifications icons for media controls
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect)
Tracking
(firefox50 verified)
VERIFIED
FIXED
Firefox 50
Tracking | Status | |
---|---|---|
firefox50 | --- | verified |
People
(Reporter: antlam, Assigned: sebastian)
References
Details
Attachments
(3 files)
In bug 1240423 we're using material icons. Since we have media controls in fennec that use our own icons, it would be visually more consistent to use our own here too. If it's not too hard, let's give these a try? Attaching icons :)
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(s.kaspari)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Flags: needinfo?(s.kaspari)
Assignee | ||
Updated•7 years ago
|
Blocks: fennec-media-control
Assignee | ||
Comment 1•7 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/58966/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/58966/
Attachment #8762079 -
Flags: review?(ahunt)
Assignee | ||
Comment 2•7 years ago
|
||
Attachment #8762080 -
Flags: feedback?(alam)
Reporter | ||
Comment 3•7 years ago
|
||
Comment on attachment 8762080 [details]
media_notification.png
<3
Attachment #8762080 -
Flags: feedback?(alam) → feedback+
Comment 4•7 years ago
|
||
Comment on attachment 8762079 [details] Bug 1279137 - Update media notification icons. https://reviewboard.mozilla.org/r/58966/#review56232 Assuming these are all optimised : )
Attachment #8762079 -
Flags: review?(ahunt) → review+
Assignee | ||
Comment 5•7 years ago
|
||
(In reply to Andrzej Hunt :ahunt from comment #4) > Assuming these are all optimised : ) Yep, I used ImageOptim on all of them. :)
Pushed by s.kaspari@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/75285ea7e4aa Update media notification icons. r=ahunt
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/75285ea7e4aa
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Comment 8•7 years ago
|
||
Verified as fixed in build 50.0a1 (2016-06-15); Device: LG G4 (Android 5.1).
Status: RESOLVED → VERIFIED
Updated•2 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
•