New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: silent print settings #31489
fix: silent print settings #31489
Conversation
4500afd
to
e8dcb53
Compare
Release Notes Persisted
|
I was unable to backport this PR to "13-x-y" cleanly; |
I was unable to backport this PR to "14-x-y" cleanly; |
I was unable to backport this PR to "16-x-y" cleanly; |
I was unable to backport this PR to "15-x-y" cleanly; |
@codebytere has manually backported this PR to "13-x-y", please check out #31618 |
Thanks @codebytere. I'm waiting for this fix a long time. Than you so much! |
i have the same question |
* fix: silent print settings * chore: update patches Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
* fix: silent print settings * chore: update patches Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
* fix: silent print settings * chore: update patches Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
* fix: silent print settings * chore: update patches Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
I think the new release with the fix come soon. |
Any ideia about the release date with this fix? |
The merged labels indicate they are now backported to 13.x, 14.x, and 15.x. So the next release for these versions will contain the fix. I do not know the official date but it could be likely in the next couple of days. In the meantime those branches can be built from source if you need it faster than the official release. |
This update is now in official releases for 14.2.1 and 15.3.1. I have tested and verified it is working as expected in both. Thanks, Electron Team! |
in my code,where is wrong? |
I'm printing using loadURL and it's ok. function print(printerName, content, widthCm, heightCm) { const html = const printers = printWindow.webContents.getPrinters(); if (!printer) { printWindow.webContents.on('did-finish-load', () => {
}); printWindow.webContents.on( printWindow.loadURL( |
@latchuk A little tip for the future. You should consider using fenced code blocks when writing code snippets inside markdown content. ```js
// WRITE YOUR CODE IN HERE
``` Else it looks like a mess and hardly readable. Enclosing code inside a fenced code block formats your code automatically for better readability. And defining the language provides you with bonus features like syntax highlighting. |
Description of Change
Closes #30889.
Closes #30340.
Closes #29334.
Closes #29286.
Fixes an issue introduced 46f3491 resultant of CL:2409467 which resulted in custom print settings not being properly propagated.
Checklist
npm test
passesRelease Notes
Notes: Fixed an issue which caused print settings to not work properly when printing silently.