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)
SeaMonkey
Themes
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9
People
(Reporter: brane, Assigned: andreww)
References
Details
Attachments
(6 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
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");
+ }
Comment 1•24 years ago
|
||
confirming, and over to themes.
Assignee: matt → hangas
Status: UNCONFIRMED → NEW
Component: Preferences → Themes
Ever confirmed: true
QA Contact: sairuh → pmac
Comment 4•24 years ago
|
||
We're willing to check in the code for this bug. Is anybody interested
in getting this fixed?
Comment 5•24 years ago
|
||
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!
Comment 6•24 years ago
|
||
Comment 7•24 years ago
|
||
Comment 8•24 years ago
|
||
Comment 9•24 years ago
|
||
Comment 10•24 years ago
|
||
Updated•24 years ago
|
Assignee | ||
Comment 11•24 years ago
|
||
Ok, I see the patches and I will try them out and get them checked in.
Assignee | ||
Comment 15•24 years ago
|
||
Assignee | ||
Comment 16•24 years ago
|
||
ready for review, I think...
Comment 17•24 years ago
|
||
r=hangas
Comment 18•24 years ago
|
||
sr=hewitt
Assignee | ||
Comment 19•24 years ago
|
||
Fix checked into the trunk.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 20•24 years ago
|
||
Marking verified on all platforms (2001-03-05-08-Mtrunk).
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Product: Core → SeaMonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•