Re: [css-grid] grid-float-001, grid-inline-float-001, grid-clear-001 and grid-inline-clear-001: review request

Le 2015-01-03 15:10, Gérard Talbot a écrit :
> Le 2014-12-18 07:01, Manuel Rego Casasnovas a écrit :
>> Hi Gérard,
>> 
>> following the line of the previous tests, I've 4 new tests to check 
>> that
>> "float" and "clear" properties don't apply to grid items.
>> 
> 
> Rego,
> 
> Sorry for the long delay.
> 
>> 
>> * grid-float-001:
>>   * Source:
>> https://github.com/mrego/csswg-test/blob/css-grid-float-clear-tests/css-grid-1/grid-model/grid-float-001.xht
>>   * View:
>> https://rawgit.com/mrego/csswg-test/css-grid-float-clear-tests/css-grid-1/grid-model/grid-float-001.xht
> 
> 1-
> The meta assert more or less repeats what is already given in the
> title element. So, you could skip the meta assert here.
> This issue has been happening in a few of your other tests too.
> 
> 2-
> You may want to create a variant of such test, say, grid-float-002
> where the code would be
> <div class="grid">
>             <div class="test-overlapping-green float-left"></div>
>             <div class="test-overlapping-green float-left"></div>
>         </div>
> 
> 3-
> You may also want to create another variant of such test where the
> left-floating box would have content, say,
> grid-float-003
> 
>   <meta content="ahem" name="flags" />
> 
> (...)
>             .grid {
>                 display: grid;
>                 font: 1.25em/2.5 Ahem; /* computes to 20px/50px */
>             }
> 
> .test-overlapping-green {
>                 background-color: green;
>             }
> 
>             .float-left {
>                 float: left;
>             }
> 
> (...)
> 
> <div class="grid">
>             <div class="test-overlapping-green float-left">FLOAT</div>
>             <div class="test-overlapping-green">NotFL</div>
>         </div>
> 
> 4-
> You may also want to create another variant of such test where the 2
> left-floating boxes have content, say,
> grid-float-004
> 
>   <meta content="ahem" name="flags" />
> 
> (...)
> 
> 
>             .grid {
>                 display: grid;
>                 font: 1.25em/2.5 Ahem; /* computes to 20px/50px */
>             }
> 
> .test-overlapping-green {
>                 background-color: green;
>             }
> 
>             .float-left {
>                 float: left;
>             }
> 
> (...)
> 
> <div class="grid">
>             <div class="test-overlapping-green float-left">FLOAT</div>
>             <div class="test-overlapping-green float-left">FLOAT</div>
>         </div>
> 
> 
> 5-
> You may also want to create another variant of such test where the 2
> left-floating boxes have not-filling content, say,
> grid-float-005
> 
>   <meta content="ahem" name="flags" />
> 
> (...)
> 
> 
>             .grid {
>                 display: grid;
>                 font: 1.25em/2.5 Ahem; /* computes to 20px/50px */
>             }
> 
> .test-overlapping-green {
>                 background-color: green;
>             }
> 
>             .float-left {
>                 float: left;
>             }
> 
> (...)
> 
> <div class="grid">
>             <div class="test-overlapping-green float-left">FL</div>
>             <div class="test-overlapping-green float-left">FLO</div>
>         </div>
> 
> 
> 6-
> You may also want to create another variant of such test where the 1
> (or 2) left-floating box(es) have not-filling content, say,
> grid-float-006
> 
>             .grid {
>                 display: grid;
>                 font: 1.25em/2.5 Ahem; /* computes to 20px/50px */
>             }
> 
> .test-overlapping-green {
>                 background-color: green;
>             }
> 
>             .float-left {
>                 float: left;
>             }
> 
> (...)
> 
> <div class="grid">
>             <div class="test-overlapping-green float-left">FLO</div>
>             <div class="test-overlapping-green">NF</div>
>         </div>
> 

Rego,

I just noticed today your

http://test.csswg.org/source/css-grid-1/test-plan/index.html

document... so maybe all I wrote was not needed or already planned by 
you.



> 
> 
>> 
>> * grid-inline-float-001:
>>   * Source:
>> https://github.com/mrego/csswg-test/blob/css-grid-float-clear-tests/css-grid-1/grid-model/grid-inline-float-001.xht
>>   * View:
>> https://rawgit.com/mrego/csswg-test/css-grid-float-clear-tests/css-grid-1/grid-model/grid-inline-float-001.xht
>> 
> 
> Same points as above for grid-inline-float-001 test.
> 
>> * grid-clear-001:
>>   * Source:
>> https://github.com/mrego/csswg-test/blob/css-grid-float-clear-tests/css-grid-1/grid-model/grid-clear-001.xht
>>   * View:
>> https://rawgit.com/mrego/csswg-test/css-grid-float-clear-tests/css-grid-1/grid-model/grid-clear-001.xht
> 
> If 'float' does not apply to grid and inline-grid elements, then how
> is it possible and realistic


<ins> to expect </ins>


> that 'clear' could (or possibly would)
> have a rendering effect on grid and inline grid elements? And so, why
> would there be a need or justification to test 'clear' property?
> 
> The safest policy would be to do those


<ins> grid versus clear </ins>


> tests. Doing those tests
> (Arron Eicholz policy) would make the coverageability of the test
> suite more complete, thorough.


---------


Other possible /support images which you may find useful to reuse for 
creating future grid tests:

pattern-* images:

http://test.csswg.org/source/css21/support/pattern-gg-gr-100x100.png

http://test.csswg.org/source/support/pattern-grg-rgr-grg.png

http://test.csswg.org/source/support/pattern-grg-rrg-rgg.png

http://test.csswg.org/source/support/pattern-rgr-grg-rgr.png

http://test.csswg.org/source/support/pattern-tr.png

and maybe

http://test.csswg.org/source/css21/support/transparent_green.png

If you use them, then you would need to create a

http://test.csswg.org/source/css-grid-1/grid-model/support/

folder and then  hg-copy those pattern-* images into such

/css-grid-1/grid-model/support/ folder.


Another /support image which could be possibly useful to reuse for your 
grid tests:

http://test.csswg.org/source/css21/support/transparent_green.png


Gérard
-- 
Test Format Guidelines
http://testthewebforward.org/docs/test-format-guidelines.html

Test Style Guidelines
http://testthewebforward.org/docs/test-style-guidelines.html

Test Templates
http://testthewebforward.org/docs/test-templates.html

CSS Naming Guidelines
http://testthewebforward.org/docs/css-naming.html

Test Review Checklist
http://testthewebforward.org/docs/review-checklist.html

CSS Metadata
http://testthewebforward.org/docs/css-metadata.html

Received on Saturday, 3 January 2015 21:24:15 UTC