Closed Bug 37700 Opened 25 years ago Closed 25 years ago

Table with floating element shrink to fit

Categories

(Core :: Layout: Tables, defect, P3)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: pierre, Assigned: buster)

References

()

Details

(Keywords: regression, testcase)

Attachments

(1 file)

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.
Marked as a regression
Keywords: regression
Attached file smaller testcase (deleted) —
note that it starts out ok and messes up when the loaded image causes a reflow. this is a dup of something..
Keywords: testcase
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
This may be a dup of bug 37414 or of bug 32581. Does the regression trace back to a new bug in incremental reflow, or does it trace back to an new incremental reflow that happens when floats are loaded? (This is a very low-priority question :P)
Reassigning to buster - Triaging Troy's bug list.
Assignee: troy → buster
this was fixed a while back
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Target Milestone: --- → M16
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.

Attachment

General

Creator:
Created:
Updated:
Size: