Closed
Bug 56201
Opened 25 years ago
Closed 25 years ago
tables not resized when incrementally reflowed due to image loading
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
mozilla0.8
People
(Reporter: jud, Assigned: karnaze)
References
()
Details
(Keywords: css2, testcase)
Attachments
(6 files)
|
683 bytes,
patch
|
Details | Diff | Splinter Review | |
|
10.29 KB,
image/png
|
Details | |
|
10.36 KB,
image/png
|
Details | |
|
1.25 KB,
image/png
|
Details | |
|
6.03 KB,
image/png
|
Details | |
|
869 bytes,
patch
|
Details | Diff | Splinter Review |
bug7243 testcase before the patch
visit the said url (I'm using the trunk release build from today) and notice
that the NS image and the cell color are spilling out over the cell boundaries.
|
Assignee | |
Comment 1•25 years ago
|
||
Reassigning to myself and marking future.
Assignee: clayton → karnaze
Target Milestone: --- → Future
|
||
Comment 3•25 years ago
|
||
Original: http://www.damowmow.com/mozilla/bugs/56201/original.html
Testcase: http://www.damowmow.com/mozilla/bugs/56201/testcase.html
Summary: cell content bleeding out of boundaries → tables not resized when incrementally reflowed due to image loading
|
|
Assignee | |
Comment 6•25 years ago
|
||
|
|
||
Comment 7•25 years ago
|
||
r=peterl
|
|
Assignee | |
Comment 8•25 years ago
|
||
The patch has been checked in.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 9•25 years ago
|
||
Changing milestone from mozilla0.9 to mozilla0.8.
Target Milestone: mozilla0.9 → mozilla0.8
|
||
Comment 10•25 years ago
|
||
|
|
||
Comment 11•25 years ago
|
||
|
|
||
Comment 12•25 years ago
|
||
|
|
||
Comment 13•25 years ago
|
||
|
|
||
Comment 14•25 years ago
|
||
this patch is bad ( or should be modified for fixed table layout) because it
violates the quote from css2
With this (fast) algorithm, the horizontal layout of the table does not depend
on the contents of the cells; it only depends on the table's
width, the width of the columns, and borders or cell spacing.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
Assignee | |
Comment 15•25 years ago
|
||
Bernd, Thanks for catching that. I'll attach a revised patch that will work the
way it used to for fixed table layout.
Whiteboard: need revised patch for fixed table layout
|
|
Assignee | |
Comment 16•25 years ago
|
||
|
|
||
Comment 17•25 years ago
|
||
sr=buster
|
|
||
Comment 18•25 years ago
|
||
r=peterl
|
|
Assignee | |
Comment 19•25 years ago
|
||
The additional patch is in.
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Whiteboard: need revised patch for fixed table layout
You need to log in
before you can comment on or make changes to this bug.
Description
•