Closed Bug 402382 Opened 12 years ago Closed 12 years ago

Remove pointless use of chrome://communicator/skin/

Categories

(Thunderbird :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3

People

(Reporter: philor, Assigned: philor)

References

()

Details

Attachments

(1 file)

Attached patch Fix v.1Splinter Review
For a toolkit app, chrome://communicator/skin/ is just chrome://global/skin/ the hard way. Even though I don't see bug 399571 building the momentum to actually remove toolkit's communicator.css that just imports global.css, we should remove our own uses, which just add confusion and an extra step as we load one file that loads another.
Attachment #287259 - Flags: review?(mkmelin+mozilla)
Blocks: 399571
Comment on attachment 287259 [details] [diff] [review]
Fix v.1

Cleanup, yey! r=me
Attachment #287259 - Flags: review?(mkmelin+mozilla) → review+
mail/themes/qute/mail/messenger.css 1.10
mail/themes/qute/editor/editor.css 1.3
mail/themes/pinstripe/mail/messenger.css 1.6
mail/themes/pinstripe/editor/editor.css 1.2
mail/base/content/customizeToolbar.xul 1.14
mail/base/content/customizeToolbarSheet.xul 1.2
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3
You need to log in before you can comment on or make changes to this bug.