Closed Bug 1669725 Opened 4 years ago Closed 3 years ago

Margins settings have no effect in pdfs

Categories

(Toolkit :: Printing, defect, P2)

defect

Tracking

()

VERIFIED FIXED
85 Branch
Tracking Status
firefox82 --- wontfix
firefox83 --- wontfix
firefox84 --- wontfix
firefox85 --- verified

People

(Reporter: asoncutean, Assigned: mstriemer)

References

(Blocks 1 open bug)

Details

(Whiteboard: [print2020_v85][old-ui-])

Attachments

(2 files)

Affected versions

  • 83.0a1 (2020-10-07)
  • 82.0b8

Affected platforms

  • Windows 10
  • macOS 10.14
  • Ubuntu 18.04

Steps to reproduce

  1. Launch Fx
  2. Make sure print.tab-modal.enabled is set on true
  3. Open any pdf (eg. https://www.irs.gov/pub/irs-pdf/f1040.pdf)
  4. Hit Ctrl + P
  5. Expand More settings
  6. Apply several Margins options from the dropdown

Expected result

  • Not exactly sure what is the desired outcome here, (I see that Chrome doesn't support this option in pdf), so either this section should be grayed out or the margins settings should be applied

Actual result

  • Margins settings takes no effect

Regression range

  • Not a regression

Suggested severity

  • S3

I guess we should just disable the option for PDFs like we do for Color when saving to PDF.

Likewise there are the headers & footer and background options which likely do nothing.

Severity: -- → S4
Priority: -- → P2
Has STR: --- → yes
Whiteboard: [print2020_v83] [old-ui-] → [print2020_v84][old-ui-]
Whiteboard: [print2020_v84][old-ui-] → [print2020_v85][old-ui-]
Assignee: nobody → mstriemer
Status: NEW → ASSIGNED
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5f2f88dae91f
Hide print settings that don't apply to PDF for PDFs r=sfoster
Attached file minimal.pdf

Looks like the pdf file that got added in the patch was a base64 encoded data-uri, rather than the raw file. Not sure if this caused the errors, but seems at least possible?

try linux/asan with an updated file_pdf.pdf
try linux/debug

Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c9bf4601f27f
Hide print settings that don't apply to PDF for PDFs r=sfoster
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

Verified fixed across platforms (Windows 10, Ubuntu 18.04, ,macOS 10.15) with Fx 85.0a1 (2020-12-14).

Status: RESOLVED → VERIFIED
Flags: needinfo?(mstriemer)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: