Closed Bug 494649 Opened 15 years ago Closed 15 years ago

Remove a few dozen unused images from Pinstripe

Categories

(Thunderbird :: Mail Window Front End, defect)

Product:
Thunderbird
Component:
Mail Window Front End
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 3.0b3

People

(Reporter: philor, Assigned: philor)

Details

Attachments

(1 file, 1 obsolete file)

Fix v.2
36.86 KB, patch
mkmelin
: review+
Details | Diff | Splinter Review
Attached patch Fix v.1 (obsolete) — — Splinter Review 
Based on the only slightly-flawed script from bug 418003 comment 11 - it potentially misses things by only considering basename, so it only knows that some tab-arrow-start.png is used, not whether it's messenger/icons/tab-arrow-start.png or messenger/tabs/tab-arrow-start.png that's used, and it gives false positives for things that are only used in C++, like new-mail-alert.png, but I'm pretty sure I whittled it down to just unused things.
Attachment #379387 - Flags: review?(mkmelin+mozilla)
Attached patch Fix v.2 — — Splinter Review 
Sigh, all that and as soon as I started looking at the qute/gnomestripe list I realized I missed icons/cancel.png
Attachment #379387 - Attachment is obsolete: true
Attachment #379388 - Flags: review?(mkmelin+mozilla)
Attachment #379387 - Flags: review?(mkmelin+mozilla)
Attachment #379388 - Flags: review?(mkmelin+mozilla) → review+
Comment on attachment 379388 [details] [diff] [review]
Fix v.2

Nice! r=mkmelin
http://hg.mozilla.org/comm-central/rev/8b99f4a8d6c6
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: