Closed
Bug 597678
Opened 16 years ago
Closed 8 years ago
Remove duplicated feed images.
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
DUPLICATE
of bug 706103
People
(Reporter: tymerkaev, Assigned: tymerkaev)
References
Details
(Keywords: memory-footprint)
Attachments
(1 file, 2 obsolete files)
|
11.22 KB,
patch
|
Details | Diff | Splinter Review |
videoFeedIcon.png and audioFeedIcon.png are duplicates of feedIcon.png.
Attachment #476517 -
Flags: review?(dao)
|
||
Comment 1•16 years ago
|
||
Comment on attachment 476517 [details] [diff] [review]
remove them
>+.feedBackground > .videoPodcastBackground,
>+.feedBackground > .audioPodcastBackground,
These selectors won't match anything, AFAIK, i.e. videoPodcastBackground and audioPodcastBackground aren't children of feedBackground.
Attachment #476517 -
Flags: review?(dao) → review-
|
|
||
Updated•16 years ago
|
Assignee: nobody → tymerkaev
Attachment #476517 -
Attachment is obsolete: true
Attachment #476518 -
Flags: review?(dao)
|
|
||
Comment 3•16 years ago
|
||
The images are also referenced here:
http://mxr.mozilla.org/mozilla-central/source/browser/components/preferences/applications.js#832
It looks like the large icons are never used, though.
Attachment #476518 -
Flags: review?(dao) → review?(gavin.sharp)
|
||
Updated•16 years ago
|
Attachment #476518 -
Flags: review?(gavin.sharp) → review?(dao)
|
|
||
Comment 4•16 years ago
|
||
Attachment #476518 -
Flags: review?(dao) → review-
Attachment #476518 -
Attachment is obsolete: true
Attachment #480614 -
Flags: review?(dao)
|
|
||
Comment 6•14 years ago
|
||
Comment on attachment 480614 [details] [diff] [review]
patch
I think it would be preferable to fix bug 706103.
Attachment #480614 -
Flags: review?(dao)
|
|
||
Comment 8•8 years ago
|
||
I think bug 706103 solved this.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(dao+bmo)
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•