W3C home > Mailing lists > Public > public-css-testsuite@w3.org > August 2010

Review Report on section 8.4 (Padding property) ~= 125 testcases

From: Gérard Talbot <css21testsuite@gtalbot.org>
Date: Sat, 21 Aug 2010 16:38:49 -0700
Message-ID: <b28abc2da70d085b04684993037a6949.squirrel@cp3.shieldhost.com>
To: "public-css-testsuite@w3.org" <public-css-testsuite@w3.org>
Cc: "Arron Eicholz" <Arron.Eicholz@microsoft.com>
Hello,

This report is mostly for Fantasai and Arron E.

I found only 2 testcases which were very incorrect:
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-013.htm
and
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-014.htm


Summary (mostly for Fantasai and Arron)
***************************************
- 13 Fractional pixel testcases from Microsoft found; I have provided an
alternative to only 2 of them. The fractional pixel problem is
definitely not easy to work around for cm unit; for other unit, it seems
to be easier. I will propose testcase replacements for the 11 remaining
testcases in another email.
- many Microsoft testcases declare div {height: 0} when this is
unneeded, unnecessary and can be safely removed
- many Microsoft testcases declare .table {border-collapse: collapse;}
when this is unneeded, unnecessary and can be safely removed
- there is no testcase, none, on inherit on percentage value and inherit
on em unit: so, this is a gap in the padding section ... and I suspect
in other sections too. I will propose 2 or 3 testcases on this later.
- I spent quite some time on
http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-003.htm
and someone else besides me will have to assess such testcase. I could
not come up with a conclusion. Fantasai or someone else familiar with
line breaking opportunities will have to make a call on that one.
- testcases with a negative padding value should all have the "invalid"
keyword in the flags list
- padding-bottom-applies-to-*.htm testcases (001 to 006) do not require
a div#wrapper and do not require a div inside the div#cell... but that
does not mean that I am rejecting those testcases
-
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-013.htm
and
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-014.htm
must be incorrect


Section 8.4 (partial) review report
===================================


Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/blocks-018.htm

I do not think this is a good test.

If the whole <span class="test"> is removed or if its padding-bottom
declaration is removed, how does the test fail? Shouldn't the test show
red?
The test is not really testing what it aims at testing (padding-bottom).

Rejected.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/blocks-019.htm

Same thing here. I do not think this is a good test.

If the whole <span class="test"> is removed or if its padding-top
declaration is removed, how does the test fail? Shouldn't the test show
red?
The test is not really testing what it aims at testing (padding-top).

Rejected.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c412-hz-box-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5502-mrgn-r-002.htm

To remove:
<link rel="help"
href="http://www.w3.org/TR/CSS21/box.html#padding-properties" title="8.4
Padding properties: 'padding-top', 'padding-right', 'padding-bottom',
'padding-left', and 'padding'">

because that testcase has very little (IMO nothing) to do with padding
property testing. If you remove all of the padding declarations (and
only rely on default padding values), then the test still passes.

That testcase should not be removed but its listing in padding section
should be removed.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5504-mrgn-l-002.htm

To remove:
<link rel="help"
href="http://www.w3.org/TR/CSS21/box.html#padding-properties" title="8.4
Padding properties: 'padding-top', 'padding-right', 'padding-bottom',
'padding-left', and 'padding'">

That testcase should not be removed but its listing in padding section
should not be.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5506-ipadn-t-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5506-ipadn-t-001.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5506-ipadn-t-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5506-padn-t-000.htm

Reviewed and approved.

A font-size 16px version of this test is available.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-001.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-003.htm

I spent quite some time on this testcase. I am not sure of my conclusion.
There is no white space collapsing occuring before rendering the block.
And
"
If a space (U+0020) at the end of a line has 'white-space' set to
'normal', 'nowrap', or 'pre-line', it is also removed.
"
but
"CSS 2.1 does not fully define where line breaking opportunities occur."

All browsers (except Firefox 3.6.8) starts writing the first 2 "x"es of
the span at position (line 1, column 9); Firefox starts such span at
position (line 2, column 0). By itself, it is not sufficient to explain
all of the differences in the layout. There are other line wrapping
occurences generated by Firefox.

Undefined.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-ipadn-r-004.htm

Suggested correction:
<meta name="flags" content="ahem image">

should be

<meta name="flags" content="ahem image invalid">

since a negative padding value will be parsed as an error.

Just like there is one in
http://test.csswg.org/suites/css2.1/20100815/html4/c5507-padn-r-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-padn-r-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-padn-r-001.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-padn-r-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5507-padn-r-003.htm

  <style type="text/css">
   div { margin: 1em; }
   .test { width: 10em; padding-right: 2em; padding-right: -5em; color:
yellow; background: navy; }
   .control { width: 12em; color: yellow; background: navy; }
  </style>

could be rewritten as

  <style type="text/css">
   div { background: navy; color: yellow; margin: 1em; }
   .test { width: 10em; padding-right: 2em; padding-right: -5em; }
   .control { width: 12em; }
  </style>


Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5508-ipadn-b-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5508-ipadn-b-001.htm

<meta name="flags" content="ahem">

should be replaced with

<meta name="flags" content="ahem invalid">

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5508-ipadn-b-002.htm

   p { margin-bottom: 3em; }
should be removed

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5508-ipadn-b-003.htm

<meta name="flags" content="ahem">

should be replaced with

<meta name="flags" content="ahem invalid">

Reviewed and approved.

A font-size 16px version of this test is available.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-ipadn-l-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-ipadn-l-001.htm

Suggested more compact code:

   .two span { padding-left: 4em; color: white; }

can be replaced with

   .two span { padding-left: 4em; }

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-ipadn-l-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-ipadn-l-003.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-ipadn-l-004.htm

Suggested correction:
<meta name="flags" content="ahem image">

should be

<meta name="flags" content="ahem image invalid">

since a negative padding value will be parsed as an error.

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-padn-l-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-padn-l-001.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-padn-l-002.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5509-padn-l-003.htm

Suggested corrections:

  <meta name="flags" content="image">
should be replaced with
  <meta name="flags" content="image invalid">

Suggested more compact code:

  <style type="text/css">
   div { margin: 1em; }
   .test { width: 10em; padding-left: 2em; padding-left: -5em; color:
yellow; background: navy; }
   .control { width: 12em; color: yellow; background: navy; }

can be replaced with

  <style type="text/css">
   div { margin: 1em; color: yellow; background: navy; }
   .test { width: 10em; padding-left: 2em; padding-left: -5em; }
   .control { width: 12em; }

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5510-ipadn-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5510-padn-000.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

.control .orange { border: solid 1em yellow; background: yellow; }

is definitely counter-intuitive!

http://test.csswg.org/suites/css2.1/20100815/html4/c5510-padn-001.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/c5510-padn-002.htm

Suggested more compact code:

   div { margin: 1em; }
   .test { width: 10em; padding: 1em; padding: -5em; color: yellow;
background: navy; }
   .control { width: 10em; border: solid 1em navy; color: yellow;
background: navy; }

can be replaced with

   div { margin: 1em; color: yellow; background: navy; }
   .test { width: 10em; padding: 1em; padding: -5em; }
   .control { width: 10em; border: solid 1em navy; }


Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-001.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-002.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-003.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-004.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-005.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-006.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-007.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-008.htm

Reviewed and approved.

[Note: padding: inherit on relative values like percentage and em have
not been tested. This is a gap in the test suite right now.]

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-001.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-002.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-003.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft


http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-001.htm

border-collapse: collapse; is not applied here and not necessary.

#row
            {
                display: table-row;
                border-collapse: collapse;


Rejected

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-005.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-006.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-007.htm

border-collapse: collapse;
is not necessary and I would remove it. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-008.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-009.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-010.htm

The expected results should indicate that there should be a bullet
(list-marker) displayed. There should be a bullet as list-marker outside
the orange border on the left hand side and inside the blue border.

"Test passes if there is space between the blue and orange lines on all
four sides. There should be a black bullet on the left-hand side of and
outside of the orange lines but inside the blue square.
"

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-011.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-012.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-013.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-014.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-015.htm

Reviewed and approved.

----------------------

Author: Ian Hickson

http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-016.htm
and
http://test.csswg.org/suites/css2.1/20100815/html4/padding-applies-to-017.htm

Rejected.

Required modification:

To replace

  <div class="table">
   <tr>
    <td>
     <span class="test"><span class="control">There should be no red on
this page.</span></span>
    </td>
   </tr>

  </div>

with

  <table>
   <tr>
    <td>
     <span class="test"><span class="control">There should be no red on
this page.</span></span>
    </td>
   </tr>

  </table>

and that will do.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-background-001.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-001.htm

div
            {
                height: 0;
            }

is not required, not needed. Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-002.htm

I think
<meta name="flags" content="">
should be
<meta name="flags" content="invalid">

For this testcase and the others following:

div
            {
                height: 0;
            }

is not required, not needed.

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-003.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-004.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-005.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-006.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-007.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-012.htm

I think
<meta name="flags" content="">
should be
<meta name="flags" content="invalid">

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-013.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-014.htm

Fractional pixel problem!

Firefox 3.6.8, Chrome 5.0.375.127 and Konqueror 4.4.5 display red caused
by the fractional pixel problem.

(#div1's padding-bottom: 1pt;)
1pt == 1.33333px in Firefox 3.6.8 which is rounded down to 1px, not to 2px.

(#div2's margin-top) -3pt == 4px in Firefox 3.6.8


[
            #div1
            {
                border-top: 2pt solid red; /* == 2px in Firefox 3.6.8 */
                padding-bottom: 1pt; /* == 1.33333px in Firefox 3.6.8
which is rounded down to 1px, not 2px */
            }
            #div2
            {
                border-bottom: 2pt solid black; /* == 2px in Firefox
3.6.8 */
                margin-top: -3pt; /* == -4px in Firefox 3.6.8 */
            }

]

12pt == 16px; 1pt == 1.33333px; 1px == 0.75pt;

To avoid elegantly that fractional pixel problem, the factor of pt unit
should be 3. E.g. 3pt, 6pt, 9pt, 12pt, etc. are good. To avoid elegantly
the reverse relation, from px to pt, a factor of 4 is good. E.g. 4px,
8px, 12px, 16px, etc. are good.

A proposed replacement

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/padding-bottom-014.htm

is available with padding-bottom: 3pt; but I do not see how the same
kind of testcase can be done with padding-bottom: 1pt;.

Rejected

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-015.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-016.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-017.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-023.htm

I think
<meta name="flags" content="">
should be
<meta name="flags" content="invalid">

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-024.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-025.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-026.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-027.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-028.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-029.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-034.htm

I think
<meta name="flags" content="">
should be
<meta name="flags" content="invalid">

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-035.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-036.htm

Fractional pixel problem!

2.54cm == 96px;

0.026458cm = 1px;

1cm = 37.79528px

0.42333cm = 16px

[
/* if set to -0.8465cm, then Konqueror 4.4.5 displays a 1px red sliver
at the bottom; */
/* if set to -0.8464cm, then Konqueror 4.4.5 does not display a 1px red
sliver at the bottom but then Chrome 5.0.x displays a 1px red sliver at
the top! */
Solving this problem with cm unit is not easy at all...
]

Best (only!) value that I can find is: 1.00542cm == 38px

Proposed suggestion/replacement:

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/padding-bottom-036.htm

Rejected.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-037.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-038.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-039.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-040.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-047.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-050.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-051.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-058.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-061.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-062.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-094.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-095.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-103.htm

Fractional pixel problem!

Rejected. These 11 testcases must be tweaked, tuned in a way that they
can avoid generating a fractional pixel problem.

I will propose testcase replacements in another email.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-001.htm
to
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-006.htm

border-collapse: collapse; declaration is not needed
div#wrapper is not necessary to such testcases. Testcases testing
padding-bottom on sub-table elements could be built without resorting to
a div#wrapper
The innermost div inside #cell is not necessary for such testcases.
Nevertheless,

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-007.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-008.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-009.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-010.htm

The testcase should state that a bullet list-marker should be rendered,
before or on the left-hand side of the orange line.

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-011.htm
http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-012.htm

Reviewed and approved.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-013.htm

The assert text says:
"The 'padding-bottom' property applies to elements with a display of
table."
but the expected results says:
"Test passes if there is no space between the blue and orange lines below."

"
The distance between the table border and the borders of the cells on
the edge of the table is the table's padding for that side.
"
http://www.w3.org/TR/CSS21/tables.html#separated-borders

Proposed replacement:

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/padding-bottom-applies-to-013.htm

Rejected.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-014.htm

The assert text says:
"The 'padding-bottom' property applies to elements with a display of
inline-table."
but the expected results says:
"Test passes if there is no space between the blue and orange lines below."

"
The distance between the table border and the borders of the cells on
the edge of the table is the table's padding for that side.
"
http://www.w3.org/TR/CSS21/tables.html#separated-borders

Proposed replacement:

http://www.gtalbot.org/BrowserBugsSection/css21testsuite/padding-bottom-applies-to-014.htm

Rejected.

----------------------

Author: Microsoft

http://test.csswg.org/suites/css2.1/20100815/html4/padding-bottom-applies-to-015.htm

Reviewed and approved.

----------------------

regards, Gérard
-- 
Contributions to the CSS 2.1 test suite:
http://www.gtalbot.org/BrowserBugsSection/css21testsuite/

CSS 2.1 test suite (beta 3; August 15th 2010):
http://test.csswg.org/suites/css2.1/20100815/html4/toc.html

CSS 2.1 test suite contributors:
http://test.csswg.org/source/contributors/
Received on Saturday, 21 August 2010 23:39:26 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Monday, 20 September 2010 17:52:03 GMT