Will need to clean up ua.css, html.css, and quirk.css (again)

NEW
Unassigned

Status

()

Core
Layout
P3
normal
17 years ago
5 years ago

People

(Reporter: Hixie (not reading bugmail), Unassigned)

Tracking

(Depends on: 3 bugs, 5 keywords)

Trunk
mozilla1.0.1
css-moz, css1, css2, css3, perf
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
History: As Netscape6 reached RTM, we performed some serious performance work
and general cleanup on the ua.css, quirk.css and html.css files. This gave us
a small performance boost, cut the file in half, and fixed a whole bunch of
lingering bugs that had easy CSS-level fixes. See bug 6625.

We will need to do this again before the next full release, be that mozilla1.0,
NS6.5, or whatever, since the ua.css file quickly accumulates cruft and is not
frequently cleaned up.

The main problem we had when doing this the previous time was that since this
is a very high risk operation (the UA stylesheets affect the entire app!),
there was a chance that the fix would not be accepted that close to release.
However, this change can _only_ happen that close to release, since otherwise
last minute changes are not caught! (To put this in perspective: between the
big changes to the UA stylesheets and the release they was aimed at, at least 4 
checkins happened, and that's with the tight check-in restrictions.)

Therefore, this is an early warning for the next time this has to occur.
Cc'ing interested parties. Please cc everyone who will be affected by this.
(Reporter)

Updated

17 years ago
Status: NEW → ASSIGNED
Keywords: css-moz, css1, css2, css3, donttest, perf
Target Milestone: --- → mozilla1.0

Comment 1

17 years ago
I think every change to ua.css and html.css should be reviewed by Pierre and/or
myself, as Style module owners. Maybe that will help keep the cruft out?

(Same for xul.css --> David Hyatt too, I believe.)

Updated

17 years ago
QA Contact: petersen → chrisd
(Reporter)

Comment 2

17 years ago
Netscape's standard compliance QA team reorganised itself once again, so taking 
remaining non-tables style bugs. Sorry about the spam. I tried to get this done 
directly at the database level, but apparently that is "not easy because of the 
shadow db", "plus it screws up the audit trail", so no can do...
QA Contact: chrisd → ian

Comment 3

17 years ago
> We will need to do this again before the next full release, be that
> mozilla1.0, NS6.5, or whatever, since the ua.css file quickly
> accumulates cruft and is not frequently cleaned up.

This needs to be done ASAP if it should be ready for Moz1.0. There are all 
sorts of freezes going on ...

Comment 4

17 years ago
Hixie, is there anything worth cleaning up in there? Seems like it has been
pretty stable overt the last 6 months, actually.
(Reporter)

Comment 5

17 years ago
Karl: Mozilla 1.0 is not likely to come out any time soon.
Marc: I haven't seen much that needs changing, no. I don't plan on doing this
for Mojo, anyway.
(Reporter)

Updated

17 years ago
Depends on: 43220
I've spotted 2 things in need of cleanup:

rule for <nobr> in html.css should be moved to quirk.css
rule for blink should be gone; if NS wants to support this for sentimental
reasons in their own build, that's their lookout.

Comment 7

16 years ago
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)
Target Milestone: mozilla1.0 → mozilla1.0.1

Comment 8

15 years ago
this bug hasn't been touched since forever. is there still a
need for this bug to be open?

related bug 84887 bug 5119 bug 15316 bug 16206 bug 33784,
bug 41806 bug 54696 bug 96966 bug 119753 bug 155507 bug 156888

performance related: bug 62892 bug 115429
(Reporter)

Updated

14 years ago
Assignee: ian → nobody
Status: ASSIGNED → NEW
QA Contact: ian → core.layout

Updated

14 years ago
Depends on: 5119, 15316, 16206, 33784, 84887
No longer depends on: 43220

Updated

14 years ago

Updated

11 years ago
No longer depends on: 156888
You need to log in before you can comment on or make changes to this bug.