1803576 - Add Save page as PDF action to QuickActions

Status: RESOLVED FIXED

(Firefox :: Address Bar, enhancement, P3)

Comment 1

[Dale Harvey (:daleharvey)]

David, are you a good person to ask about how I would trigger the print dialog to show ensuring the save to pdf option was selected? This is the code I currently have but it is going straight to the printer right now. Cheers

     let psService = Cc[
        "@mozilla.org/gfx/printsettings-service;1"
      ].getService(Ci.nsIPrintSettingsService);
      let printSettings = psService.createNewPrintSettings();
      printSettings.printerName = "Mozilla Save to PDF";
      lazy.BrowserWindowTracker.getTopWindow().gBrowser.selectedTab.linkedBrowser.browsingContext
        .print(printSettings);

Comment 2

[David Shin[:dshin]]

Hm, unfortunately don't know too much about frontend/chrome code.
That said, perhaps printSettings.outputFormat is what you want to modify? This seems relevant..

Comment 3

[Dale Harvey (:daleharvey)]

Cheers David, Mark is there any chance you know who to ask about this? (seen you on a few reviews)

(I tried the above printSettings.outputFormat but got the same results)

I can open the print dialog with

      lazy.BrowserWindowTracker.getTopWindow()
        .document.getElementById("cmd_print")
        .doCommand();

But cant find a way to define settings using that

Comment 4

[Dale Harvey (:daleharvey)]

Attachment: Bug 1803576 - Add Save page as PDF Quickaction. r=daisuke,sfoster

Comment 5

[Dale Harvey (:daleharvey)]

Clearing needinfo, sorted over chat

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/42604daaa4d1
Add Save page as PDF Quickaction. r=daisuke,fluent-reviewers,kpatenio

Comment 7

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/42604daaa4d1

Comment 8

[Donal Meehan [:dmeehan]]

:daleharvey could you consider nominating this for a release note? (Process info)

Comment 9

[Dale Harvey (:daleharvey)]

While this is enabled in nightly, we aren't looking to promote the feature as we are in discussion with rolling it into another feature and this was landed just to make sure it was included in the migration so I think we may want to skip the release notes for this particular patch.