Closed
Bug 85872
Opened 23 years ago
Closed 18 years ago
Inheritance from elements with display:table-cell broken
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
VERIFIED
FIXED
People
(Reporter: ian, Assigned: dbaron)
References
()
Details
(4 keywords, Whiteboard: [Hixie-P3][sc-frame-relationship][A2])
Attachments
(1 file)
(deleted),
text/xml
|
Details |
The 'content' property doesn't inherit from a cell to it's ':after' pseudo- element. This works fine for other elements. Testcase: http://www.hixie.ch/tests/adhoc/css/inheritance/content/002.xml
Reporter | ||
Updated•23 years ago
|
Whiteboard: [Hixie-P3]
Comment 1•23 years ago
|
||
You should have included the build id of the version you saw this on. This bug does occur on build 2001060712 on Macintosh.
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
Updated•23 years ago
|
Target Milestone: mozilla1.0 → mozilla1.0.1
Setting All/All per comment #1.
OS: Windows 2000 → All
Hardware: PC → All
Assignee | ||
Comment 3•22 years ago
|
||
Are there other inheritance problems in the same place?
Comment 4•22 years ago
|
||
Comment 5•22 years ago
|
||
So... The problem is that table cells have this block frame inside... and this block frame is what's used as the parent frame when ProcessChildren() is called from ConstructTableCellFrame(). So the pseudo style context of that block is what's used as the parent style context for the <td>'s kids.
Assignee: hyatt → dbaron
Status: ASSIGNED → NEW
Target Milestone: mozilla1.0.1 → ---
Updated•22 years ago
|
Summary: 'content' property doesn't inherit in tables [GC] [CASCADE] → Inheritance from elements with display:table-cell broken
Assignee | ||
Comment 6•22 years ago
|
||
Yet another bug resulting from the backwards relationship between style contexts and frames.
Whiteboard: [Hixie-P3] → [Hixie-P3][sc-frame-relationship]
Reporter | ||
Updated•20 years ago
|
Whiteboard: [Hixie-P3][sc-frame-relationship] → [Hixie-P3][sc-frame-relationship][A2]
Assignee | ||
Comment 7•19 years ago
|
||
I think Boris just fixed this for the dynamic case in bug 289517 but not for the static case.
Assignee | ||
Comment 8•19 years ago
|
||
Actually, I backed that part of the patch out. (And I'm actually not 100% sure it's right, although the difference is irrelevant unless somebody styles ::-moz-cell-content.)
I filed bug 323656 as table cells are not the only case when this happens
Updated•18 years ago
|
Comment 10•18 years ago
|
||
Fixed by checkin for bug 323656.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment 11•18 years ago
|
||
I don't see any :after tests in attachment 261617 [details] [diff] [review], so this testcase isn't quite redundant with tests in there and thus should be committed, correct?
Flags: in-testsuite?
Comment 12•18 years ago
|
||
I suppose. I didn't want to commit these tests as-is, because they would break if we implement the proposed CSS3 generated content stuff. I suppose we could add explicit tests that :before and :after inherit from their parent; I should write some like that.
Comment 13•18 years ago
|
||
Though that's going to take weeks, at least; if someone gets to it before me I'll be very very happy.
Comment 14•17 years ago
|
||
Verified fixed on trunk and branches. Both testcase (url and attached one) now show "PASSED" at the places where it is expected.
Status: RESOLVED → VERIFIED
Keywords: verified1.8.0.12,
verified1.8.1.4
You need to log in
before you can comment on or make changes to this bug.
Description
•