Closed Bug 244441 Opened 22 years ago Closed 17 years ago

Only one pref panel needs prefPanels.css

Categories

(SeaMonkey :: MailNews: Message Display, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: stdowa+bugzilla, Unassigned)

Details

Attachments

(1 file)

pref-windows.xul is the only real user of prefPanels.css; the other panels only rely on it to provide them with messenger/global stylesheets.
Attached patch patchSplinter Review
Attachment #149143 - Attachment description: [Untested] patch → patch
Attachment #149143 - Flags: superreview?(mscott)
Attachment #149143 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 149143 [details] [diff] [review] patch I'm sure someone told me that using @import was better than using multiple xml-stylesheet includes, but I've no idea who...
Attachment #149143 - Flags: review?(neil.parkwaycc.co.uk) → review+
Product: Browser → Seamonkey
Assignee: sspitzer → mail
BTW: prefPanels.css has because of Bug 292278 basically been reduced to: @import url("chrome://messenger/skin/"); @import url("chrome://communicator/skin/prefpanels.css"); @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"); Also pref-windows.xul has been removed as part of Bug 196272.
Comment on attachment 149143 [details] [diff] [review] patch is this patch still valid? If so, I'm happy with Neil's approval / r on this (which he has already given).
Attachment #149143 - Flags: superreview?(mscott)
Invalidated by bug 394522.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: