Crash on Print

[joshirio]

Printing a mail from ProtonMail webclient causes the entire app to crash when clicking on printing after the print preview dialog. The saved PDF file is still saved entirely on disk, so it looks like it crashes after successful printing.

The log files shows no related entry to this. Tested on Ubuntu and macOS.

[joshirio]

Another issue: the built-in (protonmail) print preview dialog cannot be closed when clicking on close or the close button (nothing happens).

[vladimiry]

@electron is known to have severe issues with printing, some recent issues:

• Printing crash the application electron/electron#19946
• Canceling the print dialog via the "print()" call in dev tools, breaks applications electron/electron#19937
• Dom frozen after cancel close window.print dialog. electron/electron#19768

I keep track of those upstream issues. Besides going to debug the case one day in order to find a workaround.

[vladimiry]

The workaround, for now, is to export the email message to EML file, import it to an email client like Thunderbird and print it there.

[vladimiry]

Electron team is making some progress on the issue. Looks like they are going to completely resolve it soon.

[BBaoVanC]

Print to PDF works fine on my MacBook (Catalina, 10.15.2). (ProtonMail 4.0 beta Web UI)
From about page:
ElectronMail v4.2.1
Electron: 7.1.10
Chromium: 78.0.3904.130
Node: 12.8.1
V8: 7.8.279.23-electron.0

[vladimiry]

@BBaoVanC does print work for you if you call it 2+ times one by one? There was an issue that printing worked only first time and then if you call it again nothing happened. They fixed it just recently in https://github.com/electron/electron/releases/tag/v7.1.11

[BBaoVanC]

Yes, it works. Tested 4 times in a row.

[vladimiry]

Thanks for testing the stuff. You were somewhat lucky getting it to work before it got officially fixed in https://github.com/electron/electron/releases/tag/v7.1.11 😄 It's a good thing but I'm still going to keep the issue open until the app updates to Electron v7.1.11+.

[vladimiry]

@joshirio, can you try this build? It comes with just released @electron version 8. I did a brief testing I didn't notice printing issues in this build.

[joshirio]

Printing actually works with v4.2.2, tested on windows and Ubuntu (snap)

@joshirio, can you try this build? It comes with just released @electron version 8

Also works fine. Looks like it was fixed in some recent update :)

[vladimiry]

Closing the issue as Electron has finally sorted the problem out.