Re: page-break-after proposed tests

Grant, Melinda wrote:
> 
>> Otherwise I think manually adding a sentence on the 
>> page-breaking tests indicating the number of expected pages 
>> would be the way to go. (I don't think we can or should do 
>> that for every test, just the page-breaking ones.)
> 
> I think that would be best.

Here's a patch for adding such a sentence to the testcases I've
already checked in.

Michael, do you want to review/approve these changes?

~fantasai
Index: page-break-after-000.xht
===================================================================
RCS file: /sources/public/CSS/CSS2.1-test-suite/cooked-tests/hp/page-break-after-000.xht,v
retrieving revision 1.1
diff -u -r1.1 page-break-after-000.xht
--- page-break-after-000.xht	5 Oct 2007 07:00:09 -0000	1.1
+++ page-break-after-000.xht	30 Oct 2007 16:04:10 -0000
@@ -15,7 +15,7 @@
 </style>
 </head>
 <body>
-	<div class="break">There must be a page break after this line of text followed by another line of content.</div>
+	<div class="break">There must be a page break after this paragraph, followed by another line of content. This test should produce two pages of output.</div>
 	<div>
 		This text should be at the top of the second page. 
 	</div>
Index: page-break-after-003.xht
===================================================================
RCS file: /sources/public/CSS/CSS2.1-test-suite/cooked-tests/hp/page-break-after-003.xht,v
retrieving revision 1.1
diff -u -r1.1 page-break-after-003.xht
--- page-break-after-003.xht	5 Oct 2007 07:00:09 -0000	1.1
+++ page-break-after-003.xht	30 Oct 2007 16:04:10 -0000
@@ -11,16 +11,13 @@
 	div {
 		page-break-after:always;
 	}
-	.dummy {
-		color:gray;
-	}
 
 </style>
 </head>
 <body>
 	<div>
 		<p>There must <strong>not</strong> be a page break after this line of text.</p>
-		<span class="dummy">Lorem ipsum dolor sit amet, consectetuer adipiscing elit.</span>
+		<span>This test should produce two pages of output.</span>
 	</div>
 	<div>
 		This text should be at the top of the second page.
Index: page-break-after-004.xht
===================================================================
RCS file: /sources/public/CSS/CSS2.1-test-suite/cooked-tests/hp/page-break-after-004.xht,v
retrieving revision 1.1
diff -u -r1.1 page-break-after-004.xht
--- page-break-after-004.xht	9 Oct 2007 02:23:17 -0000	1.1
+++ page-break-after-004.xht	30 Oct 2007 16:04:10 -0000
@@ -26,11 +26,9 @@
 </style>
 </head>
 <body>
-	<div class="noBreakAuto">There <strong>must</strong> be a page break after this line of text.</div>
-	<div class="noBreakLeft">There must <strong>not</strong> be a page break after this line of text.</div>
-	<div class="noBreakAlways">There must <strong>not</strong> be a page break after this line of text.</div>
-	<div class="dummy">
-		Lorem ipsum dolor sit amet, consectetuer adipiscing elit.
-	</div>
+	<div class="noBreakAuto">This test must produce two pages of output. This sentence must be on the first page.</div>
+	<div class="noBreakLeft">This sentence must be at the top of the second page.</div>
+	<div class="noBreakAlways dummy">Lorem ipsum dolor sit amet, consectetuer adipiscing elit.</div>
+	<p>This must be the last sentence on the second page.</p>
 </body>
-</html>
\ No newline at end of file
+</html>
Index: page-break-before-000.xht
===================================================================
RCS file: /sources/public/CSS/CSS2.1-test-suite/cooked-tests/hp/page-break-before-000.xht,v
retrieving revision 1.1
diff -u -r1.1 page-break-before-000.xht
--- page-break-before-000.xht	9 Oct 2007 02:42:09 -0000	1.1
+++ page-break-before-000.xht	30 Oct 2007 16:04:10 -0000
@@ -15,7 +15,7 @@
 </style>
 </head>
 <body>
-	<div>This text should be at the top of the first page.</div>
-	<div class="break">There must be a page break before this line of text and it should be at the top of a new page.</div>
+	<div>This test should produce two pages of output. This text should be at the top of the first page.</div>
+	<div class="break">This text should be at the top of the second page.</div>
 </body>
-</html>
\ No newline at end of file
+</html>

Received on Tuesday, 30 October 2007 16:06:39 UTC