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

Re: Tests for CSS Regions + selection submitted for review

From: Gérard Talbot <css21testsuite@gtalbot.org>
Date: Wed, 13 Feb 2013 19:42:06 -0500
Message-ID: <3027f8cf40cd4d6cc4d4e1b56b22fded.squirrel@ed-sh-cp3.entirelydigital.com>
To: "Rebecca Hauck" <rhauck@adobe.com>
Cc: "public-css-testsuite@w3.org" <public-css-testsuite@w3.org>, "Mihnea-Vlad Ovidenie" <mihnea@adobe.com>

Le Mer 13 février 2013 18:44, Rebecca Hauck a écrit :
> Hi Gerard,
> Thanks for taking the time to look at these. I probably should have
> mentioned that Regions are not fully implemented or enabled.

Now the Details field text says
"Regions are not enabled": so this is already more useful, relevant
feedback from the scripts of the test.

> To run the
> tests and see them pass, you can enable regions in Chrome in the
> about://flags and Enabling 'Experimental Webkit features'  Also, since
> these are prefixed properties, the -webkit- prefix will need to be added
> to the flow-into and flow-from properties (2 references in the .css
> file,
> one in the .js file).
> As we'll be submitting more tests soon for Regions and Exclusions (also
> requiring this additional step), how do you recommend we convey upon
> submission to the W3C? Just in the email to this list or should there be
> a
> comment in the test file?  If the latter, then the test would need to be
> updated at some point when they're fully enabled - not sure that's
> desirable.

The latter (just my opinion) ... if I understand what you're saying ...

> Either way, you should see failure if you run them in a UA where Regions
> is not implemented or enabled.

Yes: there is now a message in the Details section at the bottom of the
test page.

> This is probably why you saw them fail
> automatically. These tests are (I think) unique in that they only
> require
> you the tester to move the mouse. You needn't verify anything (I.e.,
> there
> is no "the test passes ifŠ") because once the manual execution is done,
> JS
> does the rest.

That is what I thought too. The most important/relevant test results can
only be gotten, updated once, after manual text selection ended

> As I mentioned, they're this way to use the same tests
> here
> and in Webkit.  I considered adding a sentence along the lines of "All
> of
> the tests below should pass." for additional clarity to the manual
> tester.
>  Do you think that would help?

"All of the tests below should pass.": hmm...


"Test has been completed (text selection ended) and results are now

... ?  I don't know... not sure.

> More of my comments are inline below.


>>I believe it should be
>>line 42: <div id="log"></div>
>>    <script>
>>         runSelectionTest([], true);
>>    </script>
>>    </body>
> Yep. Looks like this was a bad merge that happened in a mirrored github
> repo I'm using for these tests. Oddly, running these in the Webkit
> harness
> didn't show the issue. It's fixed now.


According to HTML validator, a script block in HTML5 can not be inserted
out of the body node like that:


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, 14 February 2013 00:42:45 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 6 January 2015 21:13:25 UTC