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

Incorrect HTML email rendering #6930

Closed
1 of 2 tasks
senchden opened this issue May 12, 2024 · 2 comments · Fixed by #6943
Closed
1 of 2 tasks

Incorrect HTML email rendering #6930

senchden opened this issue May 12, 2024 · 2 comments · Fixed by #6943
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done state:tested We tested it and are about to release it
Milestone

Comments

@senchden
Copy link

  • This is not a feature request (existing functionality does not work, not missing functionality).
    I will request features on forum or via support.
  • I've searched and did not find a similar issue.
    There is a similar issue some html emails cause rendering issues #6581 , but it was closed by adding a source viewer. As far as I understand it did not actually fix the issue.

Bug in web app, desktop and Android app

Describe the bug
Rendering of some HTML emails is broken - no readable text is visible.
I am enclosing a text file with the pseudonymized message source.

To Reproduce
Steps to reproduce the behavior:

  1. Attempt to render the html code in the enclosed file in any app - web, android or desktop.
    tuta-incorrect-display.txt

Expected behavior
The message is rendered correctly.

Screenshots
Example of incorrectly rendered email:
image
Example of expected rendering:
image

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser firefox 125

Desktop app version v227.240502.0

@senchden senchden added the bug broken functionality, usability problems, unexpected errors label May 12, 2024
@wrdhub
Copy link
Contributor

wrdhub commented May 15, 2024

Thanks for your report. Like I mentioned in the other one we decided to add a source viewer to help us figure out what is going on here, we should have properly updated the other issue to reflect that. We are still looking into this!

@wrdhub wrdhub self-assigned this May 15, 2024
wrdhub added a commit that referenced this issue May 15, 2024
linkify interacted badly with the sanitizer making some emails display as empty

close #6930
@wrdhub wrdhub added the state:done meets our definition of done label May 17, 2024
@kib42 kib42 added this to the Notifications milestone May 21, 2024
github-merge-queue bot pushed a commit that referenced this issue May 22, 2024
linkify interacted badly with the sanitizer making some emails display as empty

close #6930
@wrdhub wrdhub removed their assignment May 22, 2024
@wrdhub
Copy link
Contributor

wrdhub commented May 24, 2024

Test Notes

  • Check various plain text and html emails, make sure they display

@charlag charlag self-assigned this May 27, 2024
@charlag charlag added the state:tested We tested it and are about to release it label May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug broken functionality, usability problems, unexpected errors state:done meets our definition of done state:tested We tested it and are about to release it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants