Closed Bug 85872 Opened 22 years ago Closed 16 years ago
Inheritance from elements with display:table-cell broken
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
You should have included the build id of the version you saw this on. This bug does occur on build 2001060712 on Macintosh.
Setting All/All per comment #1.
OS: Windows 2000 → All
Hardware: PC → All
Are there other inheritance problems in the same place?
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 → ---
Summary: 'content' property doesn't inherit in tables [GC] [CASCADE] → Inheritance from elements with display:table-cell broken
Yet another bug resulting from the backwards relationship between style contexts and frames.
Whiteboard: [Hixie-P3] → [Hixie-P3][sc-frame-relationship]
Whiteboard: [Hixie-P3][sc-frame-relationship] → [Hixie-P3][sc-frame-relationship][A2]
I think Boris just fixed this for the dynamic case in bug 289517 but not for the static case.
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
Fixed by checkin for bug 323656.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
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?
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.
Though that's going to take weeks, at least; if someone gets to it before me I'll be very very happy.
Verified fixed on trunk and branches. Both testcase (url and attached one) now show "PASSED" at the places where it is expected.
You need to log in before you can comment on or make changes to this bug.