Skip to content
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

cosmetics in text version of sent mails by 2.12.2 #246

Closed
yeupou opened this issue Feb 24, 2022 · 1 comment
Closed

cosmetics in text version of sent mails by 2.12.2 #246

yeupou opened this issue Feb 24, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@yeupou
Copy link

yeupou commented Feb 24, 2022

with 2.12.2, regarding second issue described in #245:

<a href="https://www.devuan.org/">https://www.devuan.org/</a> is transformed into https://www.devuan.org/ (https://www.devuan.org/)
<a href="https://www.devuan.org/os/announce/">fork of Debian without systemd</a> that into fork of Debian without systemd (https://www.devuan.org/os/announce/)that

In the first case, the repetition of the URL is redundant (not great for links in signatures, looks odd as it is redundant)
In the second case, the removal of the space just after the </a> breaks the output
That is cosmetics.

What is more annoying is that quotes of previous messages lacks > before each line. That breaks completely showing message history.

It is the result of htmlToPlain() from snappymail/v/2.12.0/static/js/app.js

@yeupou yeupou changed the title cosmetics in text version of sent mails cosmetics in text version of sent mails by 2.12.2 Feb 24, 2022
@yeupou
Copy link
Author

yeupou commented Feb 25, 2022

Thanks!

@the-djmaze the-djmaze added the bug Something isn't working label Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants