[kmail2] [Bug 479851] Allow for a single text/html part to be saved decoded (as html file)

Paul Schwabauer bugzilla_noreply at kde.org
Wed Feb 21 13:08:35 GMT 2024


https://bugs.kde.org/show_bug.cgi?id=479851

--- Comment #4 from Paul Schwabauer <pschwabauer at intevation.de> ---
- Regarding the message view:
In `messagelib` there is already code that can save the HTML and open it with a
browser:
https://invent.kde.org/pim/messagelib/-/blob/c52d92a121d6678e96dcb8e00dbd9d61101dc67e/messageviewer/src/viewer/viewer_p.h#L488-L490
However, this code is hidden in a private class, and I can find no usage of it.

My proposal would be to expose `ViewerPrivate::slotOpenInBrowser` in `Viewer`
and add a new method `ViewerPrivate::exportToHtml` and `Viewer::exportToHtml`
similar to the `exportToPdf` methods, that can be used by kmail. This change
would require the extending of the context menu in kmail with "Export to
HTML..." and "Open in browser".

- Regarding the email/message structure view:
Currently, it is not possible to save an HTML part in the message structure
display, if the HTML is sent as a main body.
To reproduce, open an HTML mail with the HTML as main body part and press
CTRL-ALT-D to open the message structure display. Then right-click on the HTML
part.
If the HTML part is not at the top level, for example, with a
multipart/alternative mail, it is possible to save, open and view the HTML
part. However, this is not possible if the mail only consists of the HTML part.

The reason is here:
https://invent.kde.org/pim/messagelib/-/blob/c52d92a121d6678e96dcb8e00dbd9d61101dc67e/messageviewer/src/viewer/viewer_p.cpp#L1733-L1746.
To fix this, the `isTopLevel()` check has to be removed. This change should
have a minimal impact.

-- 
You are receiving this mail because:
You are the assignee for the bug.


More information about the Kdepim-bugs mailing list