- From: Gérard Talbot <css21testsuite@gtalbot.org>
- Date: Sat, 03 Jan 2015 16:23:36 -0500
- To: Manuel Rego Casasnovas <rego@igalia.com>
- Cc: Public CSS Test suite mailing list <public-css-testsuite@w3.org>
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