Closed
Bug 652447
Opened 13 years ago
Closed 13 years ago
Straighten out the weirdness of Pinstripe's livemark-item.png and livemarkItem.png
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
FIXED
Firefox 6
People
(Reporter: philor, Assigned: philor)
Details
Attachments
(1 file)
4.41 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
Winstripe and Gnomestripe have a browser/skin/livemark-item.png which is the single image of a RSS waveform over a document icon, which they use for the menu icon for items in a livemark folder. Pinstripe has a browser/skin/livemark-item.png, which contains enabled and disabled versions of that icon, plus two identical sets of the old "this page has feeds" icon from the addressbar in four phases, all of which are unused, and a browser/skin/places/livemarkItem.png which is the single image of a RSS waveform over a document icon, which it uses for the menu icon for items in a livemark folder. The ten minutes it took me to figure out what Pinstripe was doing, and the five minutes it took me to explain what it was doing, are my justification for making it stop doing it.
Attachment #528015 -
Flags: review?(dao)
Comment 1•13 years ago
|
||
Comment on attachment 528015 [details] [diff] [review] Fix v.1 Review of attachment 528015 [details] [diff] [review]: This actually belongs in places/, like gnomestripe is doing it already...
Attachment #528015 -
Flags: review?(dao) → review+
Assignee | ||
Comment 2•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/339182917a3d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 6
You need to log in
before you can comment on or make changes to this bug.
Description
•