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

RESOLVED FIXED in Thunderbird 3

Status

Thunderbird
General
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: philor, Assigned: philor)

Tracking

Trunk
Thunderbird 3

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 287259 [details] [diff] [review]
Fix v.1

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)
(Assignee)

Updated

11 years ago
Blocks: 399571

Comment 1

11 years ago
Comment on attachment 287259 [details] [diff] [review]
Fix v.1

Cleanup, yey! r=me
Attachment #287259 - Flags: review?(mkmelin+mozilla) → review+
(Assignee)

Comment 2

11 years ago
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
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3
You need to log in before you can comment on or make changes to this bug.