1654684 - Implement a paginator for the print preview

Status: VERIFIED FIXED

(Toolkit :: Printing, enhancement, P1)

Description

[Mark Striemer [:mstriemer]]

There should be a paginator overlayed onto the print preview that shows the total number of pages and allows the user to skip to the next page, previous page, beginning, or end.

This could be done in the print.html document and positioned above the preview, or possibly in the preview document itself.

Comment 1

[Blake Winton (:bwinton) (:☕️)]

(Update from Sean: "Also, FWIW, Romain and Martin agreed that the paginator can be removed from scope for 81.")

Comment 2

[Mark Striemer [:mstriemer]]

Attachment: Bug 1654684 - Show the currently previewed page and total pages in the footer WIP

Comment 4

[Jonathan Watt [:jwatt]]

FWIW nsIWebBrowserPrint's printPreviewNumPages, printPreviewCurrentPageNumber and printPreviewScrollToPage should all be working now.

Comment 5

[Blake Winton (:bwinton) (:☕️)]

[Moving front-end printing v82 bugs to v83.]

Comment 6

[Sam Foster [:sfoster] (he/him)]

It shouldn't be too much more work to get actual pagination here, but the first priority is to get a 1 of n sheets indicator on the preview.
I'm not sure how to handle duplex printing, I guess initially we'll just label those as new sheets and follow-up as necessary.

Comment 7

[Sam Foster [:sfoster] (he/him)]

Attachment: Bug 1654684 - Add sheet-indicator and preview navigation to the print preview. r?mstriemer!

Comment 8

[Sam Foster [:sfoster] (he/him)]

Attachment: Bug 1654684 - Move chevron.svg to toolkit. r?ntim

Comment 9

[Sam Foster [:sfoster] (he/him)]

I'm getting a test failure from the new browser_preview_navigation.js. It looks like there's a crash somewhere between testPreviewScroll and testPreviewNavigationCommands and the print tabmodal dialog goes away. testPreviewScroll does both focus and synthesizeKey stuff to page down the preview, as well as synthesized mouse events to scroll down. So I tried some voodoo in there to put it back into a good state but no change. Does anything suggest itself from log or the test? I'm drawing a blank...

https://treeherder.mozilla.org/jobs?repo=try&revision=6354c3f3d233b238d937f48b787ddb27f81ac669

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87d06a4d6ab6
Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
https://hg.mozilla.org/integration/autoland/rev/5a0333ca8a19
Add sheet-indicator and preview navigation to the print preview. r=mstriemer,fluent-reviewers,flod

Comment 11

[Cristian Brindusan [:cbrindusan]]

Backed out 2 changesets (bug 1654684) for bc failures at browser_ext_tabs_printPreview.
https://hg.mozilla.org/integration/autoland/rev/1c681871f691c9ffb5df3bf925154442eaf01586

Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=5a0333ca8a19b1bd0ca0080e07fcccbfcef42db7&selectedTaskRun=S7PR1aEpQyWzSJr0Y1Zhzg.0

Failure log:
https://treeherder.mozilla.org/logviewer?job_id=323746740&repo=autoland&lineNumber=31116

Comment 12

[Sam Foster [:sfoster] (he/him)]

Thanks for the backout. I can't reproduce locally yet, but am investigating.

Comment 13

[Sam Foster [:sfoster] (he/him)]

(In reply to Sam Foster [:sfoster] (he/him) from comment #9)

I'm getting a test failure from the new browser_preview_navigation.js. It looks like there's a crash somewhere between testPreviewScroll and testPreviewNavigationCommands and the print tabmodal dialog goes away.

I was able to narrow this down to testPreviewNavigationCommands, and attempted to land this with that particular test skipped (tracked in Bug 1681026). testPreviewNavigationCommands exercises the home/back/next/end buttons and doesnt fundamentally do anything unique that isn't done in the other tests, so I'm not sure yet what the problem is there.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8ae1d037aa9c
Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
https://hg.mozilla.org/integration/autoland/rev/2fdc1b5569e1
Add sheet-indicator and preview navigation to the print preview. r=mstriemer,fluent-reviewers,flod

Comment 15

[Natalia Csoregi [:nataliaCs]]

Backed out for broswer chrome failures with .printPreviewCurrentPageNumber. CLOSED TREE

backout: https://hg.mozilla.org/integration/autoland/rev/226151a97ecfe5bb2394c33c3386a118e6207eb6

push with landings: https://treeherder.mozilla.org/jobs?repo=autoland&revision=2fdc1b5569e124b2f64c4be3bc6bc10f16b217ea&group_state=expanded

failure log: (failures were triggered on subsequent pushes)

• https://treeherder.mozilla.org/logviewer?job_id=324042641&repo=autoland&lineNumber=1649
• https://treeherder.mozilla.org/logviewer?job_id=324044548&repo=autoland&lineNumber=1600

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3f892edf0139
Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
https://hg.mozilla.org/integration/autoland/rev/519c764753f3
Add sheet-indicator and preview navigation to the print preview. r=mstriemer,fluent-reviewers,flod

Comment 17

[Sam Foster [:sfoster] (he/him)]

The DeferredTask we use to debounce the scroll handling is now disarmed when the actor is destroyed (which happens when the print dialog closes) Furthermore, using the actor properties to register that scroll handler had the unintended consequence that any content scroll event would construct and enter the PrintingChild actor. We now add the scroll event listener to the contentWindow and remove it from within the actor's own lifecycle.

This was reproducing on mochitest-chrome tests on osx, so with this clean try run I'm re-landing the patch: https://treeherder.mozilla.org/jobs?repo=try&revision=2527ba2cf416bd000329340b818faa7548689d60

Comment 18

[Sam Foster [:sfoster] (he/him)]

I think we've figured out the test failure and there's a patch on bug 1681026, so I'm clearing the ni on :mstriemer.

Comment 19

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/3f892edf0139
https://hg.mozilla.org/mozilla-central/rev/519c764753f3

Comment 20

[Vlad Lucaci, QA (:vlucaci)]

Hello,

Confirming this a verified fixed on 86.0a1(20201218095607) and 85.0b3(20201217185930) with macOS 10.14.6, Win10x64 and Ubuntu 20.04