<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>64975</bug_id>
          
          <creation_ts>2011-07-21 14:00:21 -0700</creation_ts>
          <short_desc>Column spans and multi-column elements should not collapse margins with their children</short_desc>
          <delta_ts>2011-07-22 18:19:22 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Dave Hyatt">hyatt</reporter>
          <assigned_to name="Dave Hyatt">hyatt</assigned_to>
          <cc>bdakin</cc>
    
    <cc>phiw2</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>440627</commentid>
    <comment_count>0</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2011-07-21 14:00:21 -0700</bug_when>
    <thetext>Column spans and multi-column elements should establish block formatting contexts. They mostly do except for the part where they should not be collapsing margins with their children. This bug is about fixing the margin collapsing issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>440629</commentid>
    <comment_count>1</comment_count>
      <attachid>101642</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2011-07-21 14:02:30 -0700</bug_when>
    <thetext>Created attachment 101642
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>440632</commentid>
    <comment_count>2</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2011-07-21 14:17:11 -0700</bug_when>
    <thetext>Fixed in r91497. A bunch of multicol tests will need to be rebaselined. I&apos;m on Lion so can&apos;t do SL.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>441314</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Wittenbergh">phiw2</who>
    <bug_when>2011-07-22 18:19:22 -0700</bug_when>
    <thetext>Should &apos;-webkit-column-count:0&apos; establish a block formatting context and prevent margin-collapse through ?

It does with the latest nightly (r91611) on Snow leopard.
The spec says: &quot;Values must be greater than 0&quot;
http://dev.w3.org/csswg/css3-multicol/#cc

Opera 11.5 does not establish a BFC - the margins of the child element collapse through (unless of course the multi-col element has a border)

testcase:
http://dev.l-c-n.com/CSS3/multi-col/multicol-bfc3.html</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>101642</attachid>
            <date>2011-07-21 14:02:30 -0700</date>
            <delta_ts>2011-07-21 14:04:00 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>1760</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDkxNDk2KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTEtMDctMjEgIERhdmUgSHlh
dHQgIDxoeWF0dEBhcHBsZS5jb20+CisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTY0OTc1CisgICAgICAgIEVsZW1lbnRzIHRoYXQgaGF2ZSBjb2x1bW5z
IChlLmcuLCBjb2x1bW4td2lkdGggb3IgY29sdW1uLWNvdW50IG5vbi1hdXRvKSBhcyB3ZWxsIGFz
IGVsZW1lbnRzIHRoYXQKKyAgICAgICAgZXN0YWJsaXNoIGNvbHVtbi1zcGFucyBlc3RhYmxpc2gg
YmxvY2sgZm9ybWF0dGluZyBjb250ZXh0cywgYW5kIHRoZXJlZm9yZSB0aGV5IHNob3VsZCBub3Qg
YmUgYWxsb3dpbmcKKyAgICAgICAgY2hpbGQgbWFyZ2lucyB0byBjb2xsYXBzZSB3aXRoIHRoZWly
IG93biBtYXJnaW5zLgorCisgICAgICAgIENvdmVyZWQgYnkgZXhpc3RpbmcgdGVzdHMuCisKKyAg
ICAgICAgKiByZW5kZXJpbmcvUmVuZGVyQmxvY2suY3BwOgorICAgICAgICAoV2ViQ29yZTo6UmVu
ZGVyQmxvY2s6Ok1hcmdpbkluZm86Ok1hcmdpbkluZm8pOgorCiAyMDExLTA3LTE5ICBNYXR0aGV3
IERlbGFuZXkgIDxtZGVsYW5leUBhcHBsZS5jb20+CiAKICAgICAgICAgQWRkIGZhc3QgcGF0aCBm
b3IgSW1hZ2VCdWZmZXI6OmRyYXcKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5k
ZXJCbG9jay5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRl
ckJsb2NrLmNwcAkocmV2aXNpb24gOTE0NzgpCisrKyBTb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcv
UmVuZGVyQmxvY2suY3BwCSh3b3JraW5nIGNvcHkpCkBAIC05Myw3ICs5Myw4IEBAIFJlbmRlckJs
b2NrOjpNYXJnaW5JbmZvOjpNYXJnaW5JbmZvKFJlbmQKICAgICAvLyB3ZSdyZSBwb3NpdGlvbmVk
LCBmbG9hdGluZywgYSB0YWJsZSBjZWxsLgogICAgIG1fY2FuQ29sbGFwc2VXaXRoQ2hpbGRyZW4g
PSAhYmxvY2stPmlzUmVuZGVyVmlldygpICYmICFibG9jay0+aXNSb290KCkgJiYgIWJsb2NrLT5p
c1Bvc2l0aW9uZWQoKQogICAgICAgICAmJiAhYmxvY2stPmlzRmxvYXRpbmcoKSAmJiAhYmxvY2st
PmlzVGFibGVDZWxsKCkgJiYgIWJsb2NrLT5oYXNPdmVyZmxvd0NsaXAoKSAmJiAhYmxvY2stPmlz
SW5saW5lQmxvY2tPcklubGluZVRhYmxlKCkKLSAgICAgICAgJiYgIWJsb2NrLT5pc1dyaXRpbmdN
b2RlUm9vdCgpOworICAgICAgICAmJiAhYmxvY2stPmlzV3JpdGluZ01vZGVSb290KCkgJiYgYmxv
Y2stPnN0eWxlKCktPmhhc0F1dG9Db2x1bW5Db3VudCgpICYmIGJsb2NrLT5zdHlsZSgpLT5oYXNB
dXRvQ29sdW1uV2lkdGgoKQorICAgICAgICAmJiAhYmxvY2stPnN0eWxlKCktPmNvbHVtblNwYW4o
KTsKIAogICAgIG1fY2FuQ29sbGFwc2VNYXJnaW5CZWZvcmVXaXRoQ2hpbGRyZW4gPSBtX2NhbkNv
bGxhcHNlV2l0aENoaWxkcmVuICYmIChiZWZvcmVCb3JkZXJQYWRkaW5nID09IDApICYmIGJsb2Nr
LT5zdHlsZSgpLT5tYXJnaW5CZWZvcmVDb2xsYXBzZSgpICE9IE1TRVBBUkFURTsKIAo=
</data>
<flag name="review"
          id="96627"
          type_id="1"
          status="+"
          setter="bdakin"
    />
          </attachment>
      

    </bug>

</bugzilla>