[FIX]Use stylesheet disables @imported style if this uses another stylesheet [AltSS]

VERIFIED FIXED in mozilla1.0

Status

()

P2
normal
VERIFIED FIXED
19 years ago
16 years ago

People

(Reporter: burnus, Assigned: bzbarsky)

Tracking

Trunk
mozilla1.0
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(7 attachments, 2 obsolete attachments)

(Reporter)

Description

19 years ago
This is with 2000-09-08-21/Linux.

Assume this:

1. One stylesheet foo (default or alternate)
2. Another stylesheet bar (not default, only alternate), which imports foo

The result is: If you select bar (via Use stylesheet), the values from foo ar
not applyed.

If you use bar as default, everything works as expected.

This should depend on bug 47760 (body text color persists longer than stylesheet
that set it)

For alternate stylesheets in general, see bug 6782 ([RFE] UI for alternate and
user stylesheets [IMPORT])
(Reporter)

Comment 1

19 years ago
Created attachment 14381 [details]
Test-case using Screen (default) and Print (alternate)
(Reporter)

Comment 2

19 years ago
Testing showed an workaround (in the webpage). If one uses:

* stylesheet "screen.css" title="screen"
* stylesheet "screen.css" media="all" title="screen"
* alternate stylesheet "print.css" media="all" title="print"

Then switching between screen and print works (none and others work as well).
The extra first line fixes not only mozilla's problem with importing screen in
print, but also netscape 4.x since it doesn't like media. (Testcase:
http://www.physik.fu-berlin.de/~fsi/ )
(marking alternate style sheet bugs for easy tracking...)
Summary: Use stylesheet disables @imported style if this uses another stylesheet → Use stylesheet disables @imported style if this uses another stylesheet [ASS]

Updated

18 years ago
Summary: Use stylesheet disables @imported style if this uses another stylesheet [ASS] → Use stylesheet disables @imported style if this uses another stylesheet [AltSS]
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 5

17 years ago
The problem only happens when an alternate stylesheet imports another alternate 
stylesheet.  It works if it imports the default stylesheet.
Status: NEW → ASSIGNED
Target Milestone: --- → Future

Comment 6

17 years ago
Created attachment 58326 [details]
sheet0.css

Comment 7

17 years ago
Created attachment 58327 [details]
sheet1.css

Comment 8

17 years ago
Created attachment 58328 [details]
sheet2.css

Comment 9

17 years ago
Created attachment 58329 [details]
sheet3.css

Comment 10

17 years ago
Created attachment 58330 [details]
sheet2.css (updated)
Attachment #58328 - Attachment is obsolete: true

Comment 11

17 years ago
Created attachment 58331 [details]
sheet3.css (updated)
Attachment #58329 - Attachment is obsolete: true

Comment 12

17 years ago
Created attachment 58332 [details]
testcase
Created attachment 71167 [details] [diff] [review]
Patch to fix

What happened here was that we would call InsertSheetInDoc on the top-level
sheet and set it to be disabled.  Then we walked down the SheetLoadData list
and cloned the sheet for the next insertion.  So the child sheet we added was
added already disabled.

All the patch does is to make sure InsertChildSheet enables the sheet (since I
can't think of any reason child sheets would start out disabled by default).
(Assignee)

Updated

17 years ago
Priority: P3 → P2
Summary: Use stylesheet disables @imported style if this uses another stylesheet [AltSS] → [FIX]Use stylesheet disables @imported style if this uses another stylesheet [AltSS]
Target Milestone: --- → mozilla1.0

Comment 16

17 years ago
Comment on attachment 71167 [details] [diff] [review]
Patch to fix

sr=attinasi
Attachment #71167 - Flags: superreview+
Checked in on trunk for 0.9.9
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 19

16 years ago
verified 
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.