Closed Bug 515499 Opened 10 years ago Closed 10 years ago

Aero smileys

Categories

(Thunderbird :: Message Compose Window, defect)

All
Windows Vista
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0rc1

People

(Reporter: clarkbw, Assigned: andreasn)

References

Details

(Keywords: polish)

Attachments

(1 file, 1 obsolete file)

In bug 488454 we got a Vista styled format bar ( attachment 392685 [details] [diff] [review] ) but as you can see in attachment 392686 [details] we forgot to upgrade the smiley to a Vista smiley from it's Tango brother.

This doesn't block anything but would be good to see if we can make it happen in time for TB3.

Assigning to andreas to investigate.
Attached patch aero styled smileys (obsolete) — Splinter Review
Not sure how to edit the jar.nm file correctly, since it's defined in the communicator part.
Ok, I couldn't figure this out.  Asking philor for help here.  We could just replace the icons in the communicator section with this new set but that doesn't seem right.
Keywords: polish
Attached patch With packagingSplinter Review
Not a blocker, but I _wish_ it was. Tested on XP and Win7.
Attachment #403235 - Attachment is obsolete: true
Attachment #409185 - Flags: review?(dmose)
Comment on attachment 409185 [details] [diff] [review]
With packaging

Too close for missiles; I'm switching to guns.
Attachment #409185 - Flags: review?(dmose) → review?(bugzilla)
Attachment #409185 - Flags: review?(bugzilla)
Attachment #409185 - Flags: review+
Attachment #409185 - Flags: approval-thunderbird3+
http://hg.mozilla.org/comm-central/rev/e72959f54588
http://hg.mozilla.org/releases/comm-1.9.1/rev/f039c4c7ac05
Flags: in-testsuite-
Summary: vista smiley for compose format toolbar → Aero smileys
Target Milestone: --- → Thunderbird 3.0rc1
(In reply to comment #5)
> http://hg.mozilla.org/comm-central/rev/e72959f54588
> http://hg.mozilla.org/releases/comm-1.9.1/rev/f039c4c7ac05

Can this bug be closed after the checkins to comm-central and comm-1.9.1?
Oops, yeah.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.