Closed
Bug 878815
Opened 12 years ago
Closed 12 years ago
Unify Remix Icons
Categories
(Webmaker Graveyard :: Popcorn Maker, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: mjschranz, Assigned: mjschranz)
Details
Attachments
(1 file)
Make Details introduced a newer and more desirable icon for remixing. Let's use this in our embed controls as well.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #757419 -
Flags: review?(kate)
Comment 2•12 years ago
|
||
Comment on attachment 757419 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/30
Can you update this to use the font-awesome fork icon?
Attachment #757419 -
Flags: review?(kate) → review-
Assignee | ||
Comment 3•12 years ago
|
||
Is the font-awesome fork icon the same one used in the details page? http://dl.dropbox.com/u/8586348/Screenshots/le4y.png
If so, it already does. If not:
1) Where do I get it.
2) Is it an entire CSS file I have to import JUST for this one class set? Seems a bit much. At this point it matches the fork icon seen in the details bar.
Flags: needinfo?(kate)
Assignee | ||
Comment 4•12 years ago
|
||
This is what it looks like in the patch http://dl.dropbox.com/u/8586348/Screenshots/-_nb.png
Comment 5•12 years ago
|
||
I think we should close this as we have added font-awesome to popcorn and probably want to think of another way of unifying icons
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: needinfo?(kate)
Resolution: --- → WONTFIX
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•