- From: Zhang, Zhiqiang <zhiqiang.zhang@intel.com>
- Date: Fri, 9 Nov 2012 02:15:21 +0000
- To: "Gérard Talbot" <css21testsuite@gtalbot.org>
- CC: "Public CSS testsuite mailing list@w3.org" <public-css-testsuite@w3.org>, "Santos, Thiago" <thiago.santos@intel.com>, "Zhang, Haili" <haili.zhang@intel.com>, "Yu, Ling L" <ling.l.yu@intel.com>, "Yang, Lei A" <lei.a.yang@intel.com>
> Zhiqiang,
>
> One important pass/fail conditions in this test is about partially
> displayed images or clipped (sliced) images. The pass/fail conditions
> sentences should state that partially displayed cat images or sliced
> images must be considered as a test failure.
>
> Also, "2x2 cat image" may be a bit confusing to some people...
> furthermore if "image" in that sentence is singular, not plural. Hmm..
> And a single <img> could have 2 cats.
>
>
> How about
>
> Test passes if there is 2 rows of 2 cat images, with no clipped (sliced)
> cat image and no red.
>
> or
>
> Test passes if there is 2 rows of 2 cat images and if there is no
> clipped (sliced) cat image and no red.
>
>
> Another idea is to drop the reference to "image" and remove the usage of
> the word "image" in such sentence. This is furthermore relevant because
> it is often impossible to know how many <img> there are in a webpage
> without examining the source code. So,
>
>
>
> Test passes if there is 2 rows of 2 cats, with no clipped (sliced) cat
> and no red.
>
> or
>
> Test passes if there is 2 rows of 2 cats and if there is no clipped
> (sliced) cat and no red.
>
Totally agree, thank you very much for your review feedback. Updated at
https://hg.csswg.org/test/rev/2263f88caae8
Please check it and add your review line.
I'll update other tests which used cat image.
>
>
> ----------
>
>
> If the whole test only requires one element, then it is not really
> necessary to identify it with the id attribute id="test".
>
> line 10: div {
> line 27: <div></div>
>
> is okay.
>
> id="test" is best used when it is necessary to declare its counter-part:
> like id="control" or id="reference"
>
>
> Overall, this background-repeat-round-001.html test is excellent!
>
> Gérard
> --
> Contributions to the CSS 2.1 test suite:
> http://www.gtalbot.org/BrowserBugsSection/css21testsuite/
>
> CSS 2.1 Test suite RC6, March 23rd 2011:
> http://test.csswg.org/suites/css2.1/20110323/html4/toc.html
>
> CSS 2.1 test suite harness:
> http://test.csswg.org/harness/
>
> Contributing to to CSS 2.1 test suite:
> http://www.gtalbot.org/BrowserBugsSection/css21testsuite/web-authors-
> contributions-css21-testsuite.html
Received on Friday, 9 November 2012 02:15:53 UTC