Closed Bug 4519 Opened 26 years ago Closed 24 years ago

[REL POS][BLOCK]relatively positioned expanding parents (regression)

Categories

(Core :: Layout, defect, P2)

x86
Linux
defect

Tracking

()

VERIFIED FIXED
mozilla0.8

People

(Reporter: dbaron, Assigned: dbaron)

References

()

Details

(Keywords: css2, regression, Whiteboard: (py8ieh: file repaint bug if appropriate))

Attachments

(1 file)

relatively positioned elements should act for finding the height of their parent as if they were at their original position, not the relative position. This is a recent regression. The second, fifth, and seventh boxes in the above page show the problem. The yellow paragraph should overflow out of the red border in these cases. The correct behavior is hinted at in http://www.w3.org/TR/REC-CSS2/visuren.html#relative-positioning and is clearly defined in the next to last sentence of section 10.6.3: http://www.w3.org/TR/REC-CSS2/visudet.html#q17
Assignee: troy → kipp
Status: NEW → ASSIGNED
Priority: P3 → P2
Target Milestone: M6
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I just checked in a fix for this...
QA Contact: petersen → chrisd
Status: RESOLVED → VERIFIED
Using 6/3 build, verified fixed.
This has regressed again (I noticed it a few weeks ago). It happened sometime since January, I think. Reopening, and clearing target milestone.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Target Milestone: M15 → ---
Keywords: regression
The same problem also happens in the analogous tests in http://www.people.fas.harvard.edu/~dbaron/css/test/sec090302d
won't make M16
Status: REOPENED → ASSIGNED
Target Milestone: --- → M17
Summary: relatively positioned expanding parents (regression) → [RELPOS][BLOCK]relatively positioned expanding parents (regression)
I'd like to see this get in for nsbeta3. It's a pretty serious bug in our CSS2 implementation (this is the part of CSS2 "implemented" by Nav4). I would think it should be easily fixable, since it used to work...
Denying for beta3: bigger problems to solve in very limited time - sorry.
Whiteboard: [nsbeta3-]
Target Milestone: M17 → Future
Summary: [RELPOS][BLOCK]relatively positioned expanding parents (regression) → [REL POS][BLOCK]relatively positioned expanding parents (regression)
Here's a min testcase derived from dbarons page: <head> <style type="text/css"> div.contain { position: relative; margin: 100px; border: medium solid red; padding: 0; overflow: visible; } div.contain p { background: yellow; color: black; margin: 20px 0; padding: 0; border: none; } div.abs {height: 150px; } div.contain p.three { position: relative; left: 2em; font-size: 2em; } </style> </head> <body> <h1>Test of left</h1> <div class="contain"> <p class="three">This paragraph should have double the font size of the other paragraphs and should be offset double <em>its</em> font-size to the right of its normal position.</p> </div>
Attached patch proposed fix (deleted) — Splinter Review
Assigning to self, P2, mozilla0.9. Buster, could you review the patch? I haven't run the regression tests yet, but I will before I check in.
Assignee: buster → dbaron
Status: ASSIGNED → NEW
Target Milestone: Future → mozilla0.9
r=buster please add the test case from this bug to the regression tests. BTW, I've spent the last few days cleaning up the block regression tests, to be checked in soon. But you can carry on without those changes just fine.
sr=waterson
Fix checked in 2000-12-08 20:04 PST. Still need to add test case to regression tests, so leaving open for now.
Checked in testcase into block regression tests and discovered bug 66147 while writing it. Marking FIXED, although it's really only partly fixed and bug 66147 describes the rest of the problem...
Status: NEW → RESOLVED
Closed: 26 years ago24 years ago
Resolution: --- → FIXED
Target Milestone: mozilla0.9 → mozilla0.8
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
David, the test that reads "This paragraph should be offset 50px below the top of the div, but its bottom should be even with the bottom of the div." on the test page: http://www.people.fas.harvard.edu/~dbaron/css/test/sec090302a.html ...is incorrect. Is there a bug filed about the fact that on a repaint with a small dirty rect, (e.g., when scrolling) the tests that overflow their red box (e.g., tests 2, 5 and 7) don't paint? Is that bug 66147? Modulo those two issues, VERIFIED.
Status: RESOLVED → VERIFIED
Whiteboard: [nsbeta3-] → (py8ieh: file repaint bug if appropriate)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: