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

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 UTC