300960 - Content not showing up on print preview at hrcalifornia

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Martijn Wargers (dead)]

This is a follow-up of bug 270286.
A large part of the content of that page is not showing up.
I'll attach a testcase that I tried to minimize as much as possible.

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

Pagination fun...

Comment 3

[Martijn Wargers (dead)]

Attachment: reftest

This bug seems to be wfm in Firefox 22 beta.
I'm not sure if this bug would be caught with the reftest framework, but here is a reftest patch that passes on my macbook.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 751512 [details] [diff] [review]
reftest

Why do those end up the same?  Are we just ignoring the page-break-before on the <br>?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 751512 [details] [diff] [review]
reftest

I don't understand why this makes any sense...  See comments.

Comment 6

[Martijn Wargers (dead)]

(In reply to Boris Zbarsky [:bz] from comment #4)
> Comment on attachment 751512 [details] [diff] [review]
> reftest
> 
> Why do those end up the same?  Are we just ignoring the page-break-before on
> the <br>?

See http://www.w3.org/TR/CSS21/page.html#propdef-page-break-before:
"
User Agents must apply these properties to block-level elements in the normal flow of the root element. User agents may also apply these properties to other elements, e.g., 'table-row' elements. 
"

That seems a bit vague, mdn says:
https://developer.mozilla.org/en-US/docs/Web/CSS/page-break-before#Specification
"
This properties applies to block elements that generate a box. It won't apply on an empty <div> that won't generate a box.
"

Sorry for the delay.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Hmm.  Did we use to not ignore it?

Comment 8

[Martijn Wargers (dead)]

I think so, I can't remember, it happened 9 years ago.