Closed Bug 506660 Opened 11 years ago Closed 11 years ago

Need vista specific icons for Activity Manager

Categories

(Thunderbird :: General, defect)

All
Windows Vista
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b4

People

(Reporter: andreasn, Assigned: andreasn)

References

Details

Attachments

(1 file, 1 obsolete file)

Currently TB under Vista makes use of the same icons and XP. We should use Aero styled icons here instead.
The current icons are taken from #476695
Attached patch patch on trunk (obsolete) — Splinter Review
Having some issues building for the moment, so I can't show off any screenshots for the moment.
OS: Linux → Windows Vista
Hardware: x86 → All
Target Milestone: --- → Thunderbird 3.0b4
Blocks: 488060
Attachment #391089 - Flags: ui-review?(clarkbw)
Comment on attachment 391089 [details] [diff] [review]
patch on trunk

Seems like this is missing the file "defaultEventIcon-aero.png" and I think you meant to make the -aero.png icon additions at the bottom of the jar.mn file and not the top where the XP icons are. :)
Attachment #391089 - Flags: ui-review?(clarkbw) → ui-review-
Attached patch updated patchSplinter Review
Attachment #392910 - Flags: ui-review?(clarkbw)
Attachment #392910 - Flags: review?(philringnalda)
Attachment #391089 - Attachment is obsolete: true
Comment on attachment 392910 [details] [diff] [review]
updated patch

this looks good to me
Attachment #392910 - Flags: ui-review?(clarkbw) → ui-review+
Comment on attachment 392910 [details] [diff] [review]
updated patch

Nice!
Attachment #392910 - Flags: review?(philringnalda) → review+
http://hg.mozilla.org/comm-central/rev/deea9dbaebf6
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.