W3C home > Mailing lists > Public > public-css-testsuite@w3.org > November 2013

RE: TestTWF Pull requests needing review

From: Gérard Talbot <css21testsuite@gtalbot.org>
Date: Thu, 7 Nov 2013 15:17:42 -0500
Message-ID: <357dd511b54494946062140eaabf1438.squirrel@ed-sh-cp3.entirelydigital.com>
To: "Zhang, Zhiqiang" <zhiqiang.zhang@intel.com>
Cc: "Rebecca Hauck" <rhauck@adobe.com>, "Public CSS testsuite mailing list" <public-css-testsuite@w3.org>

Le Mer 6 novembre 2013 21:46, Zhang, Zhiqiang a écrit :
> From: Rebecca Hauck [mailto:rhauck@adobe.com]
> Sent: Tuesday, October 29, 2013 6:49 AM
> -------
> https://github.com/w3c/csswg-test/pull/95
> https://github.com/w3c/csswg-test/pull/97
> I've reviewed the 2 PRs on GitHub and made an improvement:
> http://zqzhang.github.io/review/box-sizing-001-review.html
> http://zqzhang.github.io/review/box-sizing-002-review.html
> http://zqzhang.github.io/review/box-sizing-001-ref-review.html
> Any comment is welcome.

[Resending this email as my webhost SMTP server seemed to have failed
sending it.]


Here's some feedback on these tests.


line 4:
<title>CSS Basic User Interface Test: box-sizing - padding-box</title>

<title>CSS Basic User Interface Test: box-sizing - padding-box


<title>CSS Basic User Interface Test: box-sizing - padding-box

line 6: <link rel="help" href="http://www.w3.org/TR/css3-ui/#box-sizing">

<link rel="help" href="http://www.w3.org/TR/css3-ui/#box-sizing"
title="6.1. 'box-sizing' property">

line 11 to 17:
        div {
            height: 200px;
            left: 10px;
            position: absolute;
            top: 50px;
            width: 200px;

setting left and top to an arbitrary value is not needed by the test and
not necessary in the test.

We already have a reftest which is very frequently used and reused for
tests which would fit perfectly here:


Shepherd indicates that such reftest is referenced by 151 tests already


So, I would change 200px (height and width) for 100px and then just
reuse that reftest. In the long term, reducing the number of reftests
brings many benefits: reusability, reduce intricability (N tests to 1
reftests instead of multiple 1 test to 1 reftest), more efficient memory

line 21:
            padding: 20px;

In other to maximize noticeability of test failure, I would set padding
to 50px.

line 30-31:
    <div class="test-overlapped-red"></div>
    <div class="ref-overlapping-green"></div>

Semantically speaking, class is for logical grouping of several
elements. Id is for linking to a document-unique element. So, here, I
would use id instead of class.

Contributions to the CSS 2.1 test suite:

CSS 2.1 Test suite RC6, March 23rd 2011:

CSS 2.1 test suite harness:

Contributing to to CSS 2.1 test suite:
Received on Thursday, 7 November 2013 20:18:16 UTC

This archive was generated by hypermail 2.4.0 : Friday, 20 January 2023 19:58:20 UTC