Closed
Bug 111849
Opened 23 years ago
Closed 23 years ago
a fixed block should become container to any children appended through DOM
Categories
(Core :: CSS Parsing and Computation, defect, P2)
Core
CSS Parsing and Computation
Tracking
()
VERIFIED
FIXED
mozilla0.9.7
People
(Reporter: alexeyc2003, Assigned: dbaron)
Details
(Keywords: dom1, testcase)
Attachments
(3 files, 2 obsolete files)
1.27 KB,
text/html
|
Details | |
2.64 KB,
patch
|
Details | Diff | Splinter Review | |
3.19 KB,
patch
|
waterson
:
review+
attinasi
:
superreview+
|
Details | Diff | Splinter Review |
Build: 2001112009 According to http://www.w3.org/TR/REC-CSS2/visuren.html#absolute-positioning An element with absolute or fixed position should become a containing block for all of it's descendants. In Mozilla, if an child is added through DOM to an absolutely positioned element the behaviour is correct. But if a child is added to a fixed element, viewport is treated as it's container. See attached testcase.
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Updated•23 years ago
|
Comment 2•23 years ago
|
||
Seeing this on Linux 2001-11-23-08 too. The DOM is correct (per DOM inspector) but looks like the frame model is confused...
OS: Windows 2000 → All
Hardware: PC → All
Assignee | ||
Comment 3•23 years ago
|
||
easy to fix
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.7
Assignee | ||
Comment 5•23 years ago
|
||
Attachment #59109 -
Attachment is obsolete: true
Assignee | ||
Comment 6•23 years ago
|
||
Assignee | ||
Comment 7•23 years ago
|
||
Assignee | ||
Updated•23 years ago
|
Attachment #59111 -
Attachment is obsolete: true
Assignee | ||
Comment 8•23 years ago
|
||
This patch makes two changes: * considersl fixed positioned elements as well, in addition to relative and absolute (fixes this bug) * walks down two scroll frames in addition to just one (although I'm leaving the just-one code since we could start from the frame where it is just one), to fix bug 91060.
Reporter | ||
Comment 9•23 years ago
|
||
Comment on attachment 59112 [details] [diff] [review] patch that also fixes bug 91060, diff -w (for review) David, I assume this patch is obsolete as well?
Assignee | ||
Comment 10•23 years ago
|
||
No, one is a diff -w and the other isn't. I just accidentally attached the wrong one when I attached the first non -w.
Comment 11•23 years ago
|
||
Comment on attachment 59113 [details] [diff] [review] patch that also fixes bug 91060 sr=attinasi
Attachment #59113 -
Flags: superreview+
Comment 12•23 years ago
|
||
Comment on attachment 59113 [details] [diff] [review] patch that also fixes bug 91060 r=waterson
Attachment #59113 -
Flags: review+
Assignee | ||
Comment 13•23 years ago
|
||
Fix checked in 2001-11-27 20:57 PDT.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•