1816086 - Mixed pages in landscape/portrait mode are printed using the mode of the first one

Status: RESOLVED DUPLICATE of bug 1833244

(Core :: Printing: Output, defect)

Description

[Calixte Denizet (:calixte)]

Attachment: portrait.html

STR:

• open the attached html and print it.

The first page should be in portrait and the second in landscape mode.
When I print to pdf, the output is a pdf with two pages in portrait mode.

If I swap the P and the L on the div styles, it's the opposite the two printed pages are in landscape mode.

The context is pdf.js: when we print a pdf with different modes we want to have the same modes for the output.

I tested in Chrome and it works correctly.

Comment 1

[Emily McDonough [:alaskanemily]]

This is currently expected behavior, as we do not support mixed page sizes yet (this was specifically disallowed by bug 1802238), although it is a bit unfortunate for some documents. We do definitely want to implement this at some point.

This will require either mixed page-size support to be fully tested enabled, or page-orientation to be fully implemented and enabled and the document to rotate the later pages using @page{ orientation: rotate-* }.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jwatt, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 3

[Daniel Holbert [:dholbert]]

I think this might be fixed -- at least I'm seeing expected results in current Nightly. I'll attach a PDF of what I'm seeing.

Comment 4

[Daniel Holbert [:dholbert]]

Attachment: output from Nightly 120.0a1 (2023-10-17)

Comment 5

[Daniel Holbert [:dholbert]]

Using mozregression --find-fix, it looks like this started working as-expected (for the attached testcase) as of bug 1833244's fix landing. --> Duping.

Comment 6

[Marco Castelluccio [:marco]]

Is it worth/feasible adding a test with the attached testcase?

Comment 7

[Daniel Holbert [:dholbert]]

Likely yes, worth adding a test. It looks like we didn't add any tests or mark any tests passing in dupe-target bug 1833244 (with one exception which isn't really passing as discussed in bug 1859823 comment 0).

I think it should be possible to morph the attached testcase into a WPT test. I'll take a look.