Closed
Bug 37700
Opened 25 years ago
Closed 25 years ago
Table with floating element shrink to fit
Categories
(Core :: Layout: Tables, defect, P3)
Core
Layout: Tables
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: pierre, Assigned: buster)
References
()
Details
(Keywords: regression, testcase)
Attachments
(1 file)
(deleted),
text/html
|
Details |
Tested with recent build on Windows and Mac:
- Go to the URL above (it's from the W3C CSS test suite)
==> The tables at the bottom of the page are shrunk to what looks to be a minimum
size.
Comment 2•25 years ago
|
||
Comment 3•25 years ago
|
||
note that it starts out ok and messes up when the loaded image causes a reflow.
this is a dup of something..
Keywords: testcase
Comment 4•25 years ago
|
||
Buster, during the incremental reflow, the cell is requesting the max
(preferred) size using NS_REFLOW_CALC_MAX_WIDTH and the block is returning too
small a value (225 twips below). CCing Troy since he implemented this capability
in the block code.
TO::Rfl en 00DE6240 rea=1 av=(8940,UC) comp=(0,0) count=10
T::Rfl en 00DE6294 rea=1 av=(8940,UC) comp=(UC,UC) count=11
TRG::Rfl 00DE6300 rea=1 av=(8910,UC) comp=(8910,UC) count=12
TR::Rfl en 00DE6340 rea=1 av=(8910,UC) comp=(8910,UC) count=13
TC::Rfl 00DE6388 rea=1 av=(8910,UC) comp=(8790,UC) count=14
Area::Rfl en 00DE63E4 rea=1 av=(8790,UC) comp=(8790,UC) count=15
Area::Rfl ex 00DE63E4 des=(8790,810) maxElem=(810,285)max=225
TC::Rfl ex 00DE6388 des=(8910,930) maxElem=(930,405)max=345
TR::Rfl ex 00DE6340 des=(8910,930) max=0
TRG::Rfl ex 00DE6300 des=(8910,0) max=0
TRG::Rfl 00DE6300 rea=2 av=(930,UC) comp=(930,UC) count=16
TR::Rfl en 00DE6340 rea=2 av=(930,UC) comp=(930,UC) count=17
TC::Rfl 00DE6388 rea=2 av=(930,UC) comp=(810,UC) count=18
Area::Rfl en 00DE63E4 rea=2 av=(810,UC) comp=(810,UC) count=19
Area::Rfl ex 00DE63E4 des=(810,6795)
TC::Rfl ex 00DE6388 des=(930,6915)
TR::Rfl ex 00DE6340 des=(930,6915)
TRG::Rfl ex 00DE6300 des=(930,6915)
T::Rfl ex 00DE6294 des=(960,6945) max=960
TO::Rfl ex 00DE6240 des=(960,6945)
Assignee: karnaze → buster
troy: any chance you could take a quick look at this and see if it's something
obvious in code you own?
Assignee: buster → troy
Comment 6•25 years ago
|
||
this was fixed a while back
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Target Milestone: --- → M16
Comment 9•24 years ago
|
||
Using 7/13 build with 5/1 testcase, verified fixed.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•