I could swear I just reviewed another very similar test, but I can't find it.
The text on the fourth page needs to be shortened so that if will all fit on a 4x6in page.
Add a help link to the css3-page spec.
Tom Clancy, 2008/11/11 13:55
Updated. There very well may be a similar test across the different groups of page break tests. If you can give me an idea of how it was similar, I can search for it.
Melinda Grant, 2008/12/03 17:55
We can't actually justify this test for 2.1, because 2.1 doesn't nail down how to determine whether the first page is a right page or a left page.
Add to the style rules for :first a 50% margin-top and change the second sentence of the first 'p' to “This content must appear entirely on the bottom half of the page.”
Melinda Grant, 2008/12/03 18:10
Oh, also need to generalize the description to allow for the first page being either a right or a left.
Tom Clancy, 2008/12/05 09:30
Ok, I've made the changes. The one thing I wasn't clear on was whether you wanted me to remove the CSS2.1 help link or if the changes now make this CSS2.1-compatible.
Melinda Grant, 2008/12/29 18:58
We're trying to make the test apply both to CSS2.1 and CSS3, so leave the help link to 2.1.
If the UA makes the first page a left page (as 2.1 allows it to do), the test will produce 5 pages of output, so please modify the description accordingly.
As is, Prince is failing the test because the style rules for :first exercise its percent height bug (even though it correctly implements :left page breaking). So let's remove the margin-top from the :first styling as it's superfluous here and is causing inadvertent failures. (Trying to remember why I asked you to add the margin-top in the last review cycle – I think it was just because the :first was there, to make it unique…) Let's change the :first styling to use equal right and left margins, since we don't know if it's a right or a left page; or remove the :first style rules completely. Change the description there to “When printed, this test produces 4 or 5 pages of output. The next page may or may not be blank.”
Also need to set the page margins: the usual 1in defaults don't leave enough room on the page for the text at 4x6in. (See Prince output with size set to 4x6in.)
Melinda Grant, 2008/12/29 20:29
Sorry, removing the :first style rules completely doesn't work because of the top-center header. So ignore that…
Tom Clancy, 2009/01/27 16:53
Updated
You could leave a comment if you were logged in.
test/css2.1/submit/119.txt · Last modified: 2014/12/09 15:48 by 127.0.0.1
Discussion
I could swear I just reviewed another very similar test, but I can't find it.
The text on the fourth page needs to be shortened so that if will all fit on a 4x6in page.
Add a help link to the css3-page spec.
Updated. There very well may be a similar test across the different groups of page break tests. If you can give me an idea of how it was similar, I can search for it.
We can't actually justify this test for 2.1, because 2.1 doesn't nail down how to determine whether the first page is a right page or a left page.
Add to the style rules for :first a 50% margin-top and change the second sentence of the first 'p' to “This content must appear entirely on the bottom half of the page.”
Oh, also need to generalize the description to allow for the first page being either a right or a left.
Ok, I've made the changes. The one thing I wasn't clear on was whether you wanted me to remove the CSS2.1 help link or if the changes now make this CSS2.1-compatible.
We're trying to make the test apply both to CSS2.1 and CSS3, so leave the help link to 2.1.
If the UA makes the first page a left page (as 2.1 allows it to do), the test will produce 5 pages of output, so please modify the description accordingly.
As is, Prince is failing the test because the style rules for :first exercise its percent height bug (even though it correctly implements :left page breaking). So let's remove the margin-top from the :first styling as it's superfluous here and is causing inadvertent failures. (Trying to remember why I asked you to add the margin-top in the last review cycle – I think it was just because the :first was there, to make it unique…) Let's change the :first styling to use equal right and left margins, since we don't know if it's a right or a left page; or remove the :first style rules completely. Change the description there to “When printed, this test produces 4 or 5 pages of output. The next page may or may not be blank.”
Also need to set the page margins: the usual 1in defaults don't leave enough room on the page for the text at 4x6in. (See Prince output with size set to 4x6in.)
Sorry, removing the :first style rules completely doesn't work because of the top-center header. So ignore that…
Updated