Closed
Bug 895686
Opened 10 years ago
Closed 10 years ago
Change - Add updated icons for contextual appbar.
Categories
(Firefox for Metro Graveyard :: Theme, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 28
People
(Reporter: jwilde, Assigned: TimAbraldes)
References
Details
(Whiteboard: feature=change c=tbd u=tbd p=1)
Attachments
(1 file)
36.16 KB,
application/zip
|
Details |
There was a new sprite format introduced with the navbar sprites in bug 811390--one icon per image, states (normal, hover, active/checked, checked hover, checked active) arranged from left to right. This format works really well. --- We're in the process of cleaning up the contextual appbar and need some more sprites: - Garbage can icon for delete (@1.0, 1.4, 1.8x) - Pin icon (@1.4, 1.8x) - Unpin icon (@1.4, 1.8x) - Restore icon (@1.4, 1.8x) - Clear selection icon (@1.4, 1.8x) For the last four, if we could get 1.0x versions in the new sprite format, that would be amazing. If you're strapped for time, I can split them myself from the original appbar-icons.png sprite in Photoshop. Thanks! :D
Updated•10 years ago
|
Priority: -- → P1
![]() |
||
Updated•10 years ago
|
Component: General → Theme
Updated•10 years ago
|
Whiteboard: [preview]
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Stephen, could we please also get a 'hide' icon for bug 903284? I think Yuan had a sketch idea for it there. Thanks!
Flags: needinfo?(shorlander)
Updated•10 years ago
|
Flags: needinfo?(shorlander)
Summary: Work - Need updated icons for contextual appbar. → Change - Add updated icons for contextual appbar.
Whiteboard: [preview] → feature=change c=tbd u=tbd p=0
Assignee | ||
Comment 3•10 years ago
|
||
I might as well fix this as part of bug 903284
Assignee: shorlander → tabraldes
Depends on: 903284
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Comment 4•10 years ago
|
||
Hey Tim, can you provide a point value.
Flags: needinfo?(tabraldes)
Priority: P1 → P2
QA Contact: jbecerra
Updated•10 years ago
|
Whiteboard: feature=change c=tbd u=tbd p=0 → feature=change c=tbd u=tbd p=1
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/9e45b2101381
Whiteboard: feature=change c=tbd u=tbd p=1 → feature=change c=tbd u=tbd p=1[fixed-in-fx-team]
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9e45b2101381
Comment 8•10 years ago
|
||
Please don't resolve bugs unless you land directly on m-c. We have scripts that mark the bugs automatically.
Target Milestone: --- → Firefox 28
Updated•10 years ago
|
Whiteboard: feature=change c=tbd u=tbd p=1[fixed-in-fx-team] → feature=change c=tbd u=tbd p=1
Comment 9•10 years ago
|
||
Verified as fixed, for iteration #20, with latest Nightly (build ID: 20131209053402) on Win 8 64-bit: I can see all icons for contextual appbar, as shown in https://bugzilla.mozilla.org/attachment.cgi?id=805542
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•