Closed Bug 58747 Opened 24 years ago Closed 24 years ago

Smiley face not shown in "Message Display" prefs in classic theme

Categories

(SeaMonkey :: Themes, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9

People

(Reporter: brane, Assigned: andreww)

References

Details

Attachments

(6 files)

There's a buglet in the Classic theme: the prefpanels.css stylesheet is missing from the skin, so the smiley face isn't shown in the "Message Display" pane. I just copied the file and binding from the Blue theme. Here's a patch. diff -rc bin/chrome/classic/skin/classic/communicator/communicator.css Seamonkey/chrome/classic/skin/classic/communicator/communicator.css *** bin/chrome/classic/skin/classic/communicator/communicator.css Wed Aug 23 18:38:06 2000 --- Seamonkey/chrome/classic/skin/classic/communicator/communicator.css Wed Nov 1 17:12:51 2000 *************** *** 13,18 **** --- 13,19 ---- @import url(chrome://communicator/skin/brand.css); @import url(chrome://communicator/skin/menubutton.css); @import url(chrome://communicator/skin/formatting.css); + @import url(chrome://communicator/skin/prefpanels.css); @import url(chrome://communicator/skin/toolbar.css); /* should we put this here, or make a dialog include it specifically? */ @import url(chrome://communicator/skin/search-widgets.css); diff -cP bin/chrome/classic/skin/classic/communicator/prefpanels.css Seamonkey/chrome/classic/skin/classic/communicator/prefpanels.css *** bin/chrome/classic/skin/classic/communicator/prefpanels.css Thu Jan 1 01:00:00 1970 --- Seamonkey/chrome/classic/skin/classic/communicator/prefpanels.css Wed Nov 1 16:29:26 2000 *************** *** 0 **** --- 1,21 ---- + /********** Fonts ***************/ + + #sizeVar, #sizeMono { + width: 4em; + } + + #mouseWheelMode { + width: 9em; + } + + /******* SmartBrowsing **********/ + + #disabledDomains { + max-height: 13em; + } + + + .smileface + { + list-style-image : url("chrome://messenger/skin/smile.gif"); + }
confirming, and over to themes.
Assignee: matt → hangas
Status: UNCONFIRMED → NEW
Component: Preferences → Themes
Ever confirmed: true
QA Contact: sairuh → pmac
Sending to Andrew
Assignee: hangas → andreww
*** Bug 55002 has been marked as a duplicate of this bug. ***
Status: NEW → ASSIGNED
We're willing to check in the code for this bug. Is anybody interested in getting this fixed?
jessica, that'd be great! pls attach a patch to this bug and add the patch/review keywords to get onto someone's radar for review/checkin... thx!
Keywords: patch, review
Ok, I see the patches and I will try them out and get them checked in.
Priority: P3 → P2
Themes Triage Team nsbeta1+
Keywords: nsbeta1
setting milestone.
Target Milestone: --- → mozilla0.8
Moving to 0.9
Target Milestone: mozilla0.8 → mozilla0.9
ready for review, I think...
r=hangas
sr=hewitt
Fix checked into the trunk.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Marking verified on all platforms (2001-03-05-08-Mtrunk).
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: