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

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

From: fantasai <fantasai.lists@inkedblade.net>
Date: Thu, 09 Sep 2010 18:42:17 -0700
Message-ID: <4C898CF9.6000706@inkedblade.net>
To: css21testsuite@gtalbot.org
CC: "public-css-testsuite@w3.org" <public-css-testsuite@w3.org>
On 09/09/2010 05:44 PM, "Gérard Talbot" wrote:
>
>
>> Fixed.
>> http://test.csswg.org/source/approved/css2.1/src/css1/c5502-mrgn-r-002.htm
>
>
> I missed the missing<meta>  flags and missing<meta>  assert in that test.
>
> Proposed addition:
>
>   <meta name="flags" content="should">
>   <meta name="assert" content="">

Added the flag. There is no need to add an empty assert -- if there's no
assert, the meta shouldn't be there at all.

> -----------------
>
>
>>>      div { margin: 1em; color: yellow; background: navy; }
>>>      .test { width: 10em; padding: 1em; padding: -5em; }
>>>      .control { width: 10em; border: solid 1em navy; }
>>
>> Fixed.
>> http://test.csswg.org/source/approved/css2.1/src/css1/c5510-padn-002.htm
>
>
> Proposed furthermore compact code:
>
>    div { background: navy; color: yellow; margin: 1em; width: 10em; }
>    .test { padding: 1em; padding: -5em; }
>    .control { border: solid 1em navy; }

I think I'm going to skip this tweak, if you don't mind. :)
I think it's readable enough with the colors pulled out.

~fantasai
Received on Friday, 10 September 2010 01:42:53 GMT

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