-
Notifications
You must be signed in to change notification settings - Fork 255
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
Enh/Improve email printing #9669
Conversation
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
I think a Print preview would be the easiest way around either a pop up or a new page. |
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
@hamza221 I found a solution to be overlaying an Having said that, I'm open to any other less hacky solutions anyone might have |
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
I am not confident with the way this works. Building HTML by hand is scary. Why do we need this separate page? Can't we use the existing page and adjust the layout for print? E.g. do not limit the height of messages in the thread when printing so that everything is included, hide unwanted elements, etc. |
We can also use the already existing IFrame but we can't include the sender's address... |
Fix #8866
TODO: