On mac, system modal has the “All” radio button selected even if a custom page range was set for "Save to pdf" option
Categories
(Toolkit :: Printing, defect, P2)
Tracking
()
People
(Reporter: emmamalysz, Unassigned)
References
Details
(Whiteboard: [print2020_v86] )
STR:
- Open firefox with new print preview ui
- Choose "Save to pdf" printer
- Specify page range
- Open system dialog
ER:
-From radio button is selected with correct range
AR:
-All radio button is selected, page range is incorrect
![]() |
||
Comment 1•4 years ago
|
||
This appears to be both a bug in the frontend code, and a bug in the platform code.
The nsIPrintSettings object passed to nsPrintDialogServiceX::Show does not have kRangeSpecifiedPageRange
set on it (or startPageRange
and endPageRange
set). That said, if I fix up the settings object the nsPrintDialogServiceX code still doesn't pick up those settings, so we need another bug for the platform work to make that happen.
![]() |
||
Comment 2•4 years ago
|
||
I must be wrong about there being a frontend bug since this works on Windows. I'll dig a bit later as to why that is.
Reporter | ||
Comment 3•4 years ago
|
||
There does appear to be a frontend bug with: https://searchfox.org/mozilla-central/rev/7ef5cefd0468b8f509efe38e0212de2398f4c8b3/toolkit/components/printing/content/print.js#219-222
this.settings
holds the correct printRange/startRange/endRange, but since it's a save to pdf printer, we're using the default system printer settings.
It appears we had written that to avoid problems with macOS that didn't have printers, but I think we should persist these settings
Updated•4 years ago
|
Updated•4 years ago
|
Comment 4•4 years ago
|
||
It seems that I can reproduce the issue with Firefox84.0b4 (20201122152513) and Firefox 85.0a1 (20201122213806) on Windows 10x64 and Windows 7x64 as well by following the STR from comment 0.
Reporter | ||
Comment 6•4 years ago
|
||
A fix for Bug 1663005 should cover this
Updated•4 years ago
|
Updated•4 years ago
|
![]() |
||
Updated•3 years ago
|
Description
•