W3C home > Mailing lists > Public > public-css-testsuite@w3.org > September 2007

Re: page-break-after proposed tests

From: fantasai <fantasai.lists@inkedblade.net>
Date: Tue, 25 Sep 2007 12:25:26 -0400
Message-ID: <46F93676.4090605@inkedblade.net>
To: public-css-testsuite@w3.org

Sorry for taking so long to get to the review; I was out of
the country for most of this month.

First comment: These files should all be .xht, as they are
XHTML and must be served as such, not as text/html.

 > page-break-after: always

Two comments:
   - The second div shouldn't be marked as dummy text anymore.
   - The first paragraph should say something about there being
     a sentence after the page break: if the UA breaks correctly
     but only prints the first page, that needs to be a fail.

One nit:
   - Add a space between 'page-break-after:' and 'always' in the <title>

 > page-break-after: auto

I suggest changing the dummy text to read "This sentence must be
on the first page." (and making it non-dummy text)

 > page-break-after: (invalid syntax)

This test is incorrect.
   - The first paragraph should say that there *must* be a page break after it.
   - Either
       a) the first style declaration rule for the second paragraph should say
            page-break-after: auto;
          instead of
            page-break-after: always;
       b) the second paragraph should say that there must be a page break
          after it.

BTW, did you run your tests in Prince to see if they pass? Because this
one, as written, should have failed.

 > page-break-after: (inline elements)

Same comments as for t130301-page-break-after-auto-02-a.html, otherwise
it looks good.

 > page-break-after: (no inherit)

Looks good.

Received on Tuesday, 25 September 2007 16:25:54 UTC

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