Make our css more efficient and remove some obsolete style rules

RESOLVED FIXED in seamonkey2.23

Status

SeaMonkey
Themes
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: stefanh, Assigned: stefanh)

Tracking

Trunk
seamonkey2.23
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

4 years ago
I noticed bug 915909, comment #9 - so I figured a file bug for this. I've also noticed a bunch of obsolete rules.
(Assignee)

Comment 1

4 years ago
Created attachment 804922 [details] [diff] [review]
v1.0

Not yet tested. I found quite a few obsolete rules...
(Assignee)

Comment 2

4 years ago
Created attachment 804929 [details] [diff] [review]
v1.1

I removed some obsolete rules. When it comes to the window.dialog rule and the description.error rule in communicator/profile/profile.css i figured the padding wouldn't have been nice if it had worked and I didn't understood the description.error rule (I can put it back, but I don't see why it should be a different colour).

dialog.css is now empty in classic since I couldn't find any .push menus (I guess -moz-appearance would overrule some of the rules if we had .push menus)
Attachment #804922 - Attachment is obsolete: true
Attachment #804929 - Flags: superreview?(neil)
Attachment #804929 - Flags: review?(neil)
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED

Comment 3

4 years ago
(In reply to Stefan from comment #2)
> I removed some obsolete rules. When it comes to the window.dialog rule and
> the description.error rule in communicator/profile/profile.css i figured the
> padding wouldn't have been nice if it had worked and I didn't understood the
> description.error rule (I can put it back, but I don't see why it should be
> a different colour).
window.dialog was obsolete since a Ben Goodger change on 2000-03-28, but description.error is still a real class and should be kept (actually it should be added somewhere else too so that pref-directory-add.xul can use it.)

> dialog.css is now empty in classic since I couldn't find any .push menus (I
> guess -moz-appearance would overrule some of the rules if we had .push menus)
I can only guess that .push menus were obsoleted by button type="menu".
(Assignee)

Comment 4

4 years ago
Created attachment 805002 [details] [diff] [review]
v1.2

I added the description.error rule in communicator.css (messenger.css imports communicator.css and profileSelection.xul uses communicator.css)

The other option is to put the rule back in profile.css and add it in messenger.css (used by pref-directory-add.xul).
Attachment #804929 - Attachment is obsolete: true
Attachment #804929 - Flags: superreview?(neil)
Attachment #804929 - Flags: review?(neil)
Attachment #805002 - Flags: superreview?(neil)
Attachment #805002 - Flags: review?(neil)
(Assignee)

Comment 5

4 years ago
Hmm, profileSelection.xul have this:
<description id="intro" style="width: 17em;"

and profile.css has this:
#intro {
  width: 17em;
}

There are some other hardcoded style rules in that dialog that looks strange. I can open a separate bug for that.
(Assignee)

Comment 6

4 years ago
Comment on attachment 805002 [details] [diff] [review]
v1.2

Oh, I see that I missed treecell.importsampledata which does looks like it could be used without the element name.
Attachment #805002 - Flags: superreview?(neil)
Attachment #805002 - Flags: review?(neil)
(Assignee)

Comment 7

4 years ago
Created attachment 805006 [details] [diff] [review]
Final version

See comment #4 and comment #6.
Attachment #805002 - Attachment is obsolete: true
Attachment #805006 - Flags: superreview?(neil)
Attachment #805006 - Flags: review?(neil)
(Assignee)

Updated

4 years ago
OS: Mac OS X → All

Comment 8

4 years ago
Comment on attachment 805006 [details] [diff] [review]
Final version

Looks like the syncIcon element got removed in bug 684537 (port of bug 626949) but the CSS was forgotten about.
Attachment #805006 - Flags: superreview?(neil)
Attachment #805006 - Flags: superreview+
Attachment #805006 - Flags: review?(neil)
Attachment #805006 - Flags: review+
(Assignee)

Comment 9

4 years ago
http://hg.mozilla.org/comm-central/rev/58c86e8a42fb

Then I noticed an error in one of the "error messages" comments which I fixed by pushing
http://hg.mozilla.org/comm-central/rev/40c0e9b4ab21 (yes, I messed up the commit msg a bit)
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.23
Comment on attachment 805006 [details] [diff] [review]
Final version

>-<?xml-stylesheet href="chrome://messenger/skin/dialogs.css" type="text/css"?>
>-
Oops, it was wrong to remove these completely, instead just the dialogs.css should have been removed, leaving chrome://messenger/skin/. This broke message print preview for example.
(Assignee)

Updated

4 years ago
Depends on: 918925
You need to log in before you can comment on or make changes to this bug.