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

Re: CSS3-writing-modes: text-orientation

From: Gérard Talbot <css21testsuite@gtalbot.org>
Date: Thu, 14 Feb 2013 20:02:56 -0500
Message-ID: <8c872fd5e06e8a06454742629fb0efca.squirrel@ed-sh-cp3.entirelydigital.com>
To: "taka" <takaoshiyama@gmail.com>
Cc: public-css-testsuite@w3.org

Le Jeu 14 février 2013 10:50, taka a écrit :
> Hi
> I appreciate your kind review for the subject test cases I have
> submitted.
> test cases: text-orientation-010, 011, 012, 014.xht
> reference files: text-orientation-010, 011, 012, 014-ref.xht
> 	http://test.csswg.org/shepherd/search/spec/css3-writing-modes/author/taka/status/submitted/
> regards,


line 5:
<title>CSS3 Writing Modes Test: text-combine-horizontal</title>

I suggest

<title>CSS Writing Modes Test: text-combine-horizontal - none</title>

Nota bene: it should be CSS and not CSS3; we will want to reuse this
test if/when there will be a CSS4 writing mode specification.

One good method of writing title text in a consistent and predictable
manner is:

<title>CSS [Name_of_module] Test: [property_name] - [property_value]
[optional: some_additional_parameters]</title>


line 11:
<meta name="flags" content="" />

Since your tests are requiring special, specific fonts, do not forget to
add the font flag.


line 12:
<meta name="assert" content="This property allows the combination of
multiple characters into the space of a single character ." />

The assert text describes the goal of the test, what it is testing.

One good method of writing the text assert in a consistent and
predictable manner is:

<meta name="assert" content="This test checks, verifies that
[property_name]: [property_value] with [some_additional_parameters] does
[expected_results]." />

So, here, I would suggest:

<meta name="assert" content="This test checks, verifies that
'text-combine-horizontal: none' does not combine multiple characters
into the space of a single character." />


line 39:
text-orientaion: mixed;
should be
text-orientation: mixed;


line 50:
<span class="tcy">1</span>&#26376;<span
/><span class="tcy">1</span>&#26376;<span class="tcy">3</span>

Maybe I'm wrong here but... why is there no span with more than 1
character.. Shouldn't that be necessary in the test? Shouldn't that be
what would be need to be tested?


line 50:
class attribute versus id attribute

Class attribute is best used for logical grouping of many elements;
semantically speaking, this was the genuine design intent for HTML.

id attribute is best used for one single element in a document.

So, a bit better is

<div id="test"> ... </div>

<div id="control"> ... </div>

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 Friday, 15 February 2013 01:03:34 UTC

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