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

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>16049</bug_id>
          
          <creation_ts>2012-02-20 21:41:30 +0000</creation_ts>
          <short_desc>Margin collapsing: clarification needed to Note in 10.7 (min-height and max-height)</short_desc>
          <delta_ts>2012-05-21 16:22:11 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>CSS</product>
          <component>CSS Level 2</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>ASSIGNED</bug_status>
          <resolution></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="Anton P">antonsforums</reporter>
          <assigned_to name="Bert Bos">bert</assigned_to>
          
          
          <qa_contact>public-css-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>64341</commentid>
    <comment_count>0</comment_count>
    <who name="Anton P">antonsforums</who>
    <bug_when>2012-02-20 21:41:30 +0000</bug_when>
    <thetext>Reported by Anton Prowse.

At the F2F in Paris 2012
[http://lists.w3.org/Archives/Public/www-style/2012Feb/0529.html] there was a discussion about the note in 10.7 (min-height and max-height) which appears immediately after the algorithm describing the calculation of a tentative height and then used height (based on min-height and max-height).  The Note says:

  # These steps do not affect the real computed values of the above
  # properties. The change of used &apos;height&apos; has no effect on margin
  # collapsing except as specifically required by rules for &apos;min-height&apos;
  # or &apos;max-height&apos; in &quot;Collapsing margins&quot; (8.3.1).

There was strong agreement that the second sentence is intended to mean that you determine whether and which margins collapse during the calculation of the tentative height, and that information remains fixed during any recalculation of the used height based on the values of min-height and max-height.  That is, you don&apos;t recompute anything with respect to margin collapsing.

I feel that this is not clearly captured by the Note.

Additionally, the mention of &apos;max-height&apos; is confusing since there are no longer any rules concerning &apos;max-height&apos; in 8.3.1 (since the spec changed a while ago, removing any special consideration of &apos;max-height&apos;).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64342</commentid>
    <comment_count>1</comment_count>
    <who name="Anton P">antonsforums</who>
    <bug_when>2012-02-20 21:41:44 +0000</bug_when>
    <thetext>Proposal

Replace:

  # These steps do not affect the real computed values of the above
  # properties. The change of used &apos;height&apos; has no effect on margin
  # collapsing except as specifically required by rules for
  # &apos;min-height&apos; or &apos;max-height&apos; in &quot;Collapsing margins&quot; (8.3.1).

with:

  | These steps do not affect the real computed values of the above
  | properties. The change of used &apos;height&apos; has no effect on margin
  | collapsing; the real computed value of &apos;height&apos; is used to
  | determine which margins are adjoining in &quot;Collapsing margins&quot;
  | (8.3.1) during these steps.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>66615</commentid>
    <comment_count>2</comment_count>
    <who name="Anton P">antonsforums</who>
    <bug_when>2012-04-11 15:54:30 +0000</bug_when>
    <thetext>An updated proposal is: [http://lists.w3.org/Archives/Public/www-style/2012Mar/0052.html]

Replace:
   # These steps do not affect the real computed values of the above
   # properties. The change of used &apos;height&apos; has no effect on margin
   # collapsing except as specifically required by rules for
   # &apos;min-height&apos; or &apos;max-height&apos; in &quot;Collapsing margins&quot; (8.3.1).

with:

   | These steps do not affect the real computed value of &apos;height&apos;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>66617</commentid>
    <comment_count>3</comment_count>
    <who name="Anton P">antonsforums</who>
    <bug_when>2012-04-11 16:01:30 +0000</bug_when>
    <thetext>The WG expressed an interest in keeping some sort of note about margin collapsing.  A possible second sentence could be:

# Consequently, for example, they do not affect margin collapsing, which depends on the computed value.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>67970</commentid>
    <comment_count>4</comment_count>
    <who name="Anton P">antonsforums</who>
    <bug_when>2012-05-21 16:22:11 +0000</bug_when>
    <thetext>The WG resolved to accept the following change to close this issue.

Replace:
  # These steps do not affect the real computed values of the above
  # properties. The change of used &apos;height&apos; has no effect on margin
  # collapsing except as specifically required by rules for
  # &apos;min-height&apos; or &apos;max-height&apos; in &quot;Collapsing margins&quot; (8.3.1).

with:
  | These steps do not affect the real computed value of &apos;height&apos;.
  | Consequently, for example, they do not affect margin collapsing,
  | which depends on the computed value.


Minutes of resolution:
http://lists.w3.org/Archives/Public/www-style/2012Apr/0276.html
http://lists.w3.org/Archives/Public/www-style/2012Apr/0101.html</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>