Closed Bug 391979 Opened 18 years ago Closed 18 years ago

-moz-column-width in "ch" units triggers "ASSERTION: column-count and column-width can't both be auto"

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9beta3

People

(Reporter: jruderman, Assigned: MatsPalmgren_bugz)

References

Details

(Keywords: assertion, css3, testcase)

Attachments

(3 files)

Attached file testcase
###!!! ASSERTION: column-count and column-width can't both be auto: 'colStyle->mColumnCount > 0', file /Users/jruderman/trunk/mozilla/layout/generic/nsColumnSetFrame.cpp, line 345
Attached patch Patch rev. 1Splinter Review
Make nsColumnSetFrame deal with 'ch' unit for -moz-column-width/gap.
Assignee: nobody → mats.palmgren
Status: NEW → ASSIGNED
Attachment #293370 - Flags: superreview?(roc)
Attachment #293370 - Flags: review?(roc)
Flags: in-testsuite?
Keywords: css3
OS: Mac OS X → All
Hardware: PC → All
Blocks: 392115
Attached patch reftest.diffSplinter Review
Attachment #293370 - Flags: superreview?(roc)
Attachment #293370 - Flags: superreview+
Attachment #293370 - Flags: review?(roc)
Attachment #293370 - Flags: review+
Comment on attachment 293370 [details] [diff] [review] Patch rev. 1 Low risk. Only affects multi-column layout.
Attachment #293370 - Flags: approval1.9?
Comment on attachment 293370 [details] [diff] [review] Patch rev. 1 a=beltzner for 1.9
Attachment #293370 - Flags: approval1.9? → approval1.9+
mozilla/layout/generic/nsColumnSetFrame.cpp 3.46 mozilla/layout/reftests/bugs/391979.html 1.3 mozilla/layout/reftests/bugs/391979-ref.html 1.3 mozilla/layout/reftests/bugs/reftest.list 1.308 -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M11
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: