-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add orphan inline images below the message body #9326
Conversation
* In plaintext mode: all inline images will be displayed below the message body * In HTML mode: all inline images that are **not** referenced in the message HTML body will be listed below the message body This doesn't add more CPU usage as it is using the already called HTML filtering function to discover if an inline part is referenced or not. This fixes issue roundcube#5051 and is a bit better than roundcube#9150
Not sure why CI is failing, but it's not on any code I changed in this PR. |
I think this is wrong because you display the images even if |
@alecpl As far as I understand the code of Roundcube, it is not possible to do what you want currently as the message body is parsed after the display of the attachments, so there is no way to know if the message parts are not in the message body. This is why I'm appending the parts below the message and not in the message list. Please suggest a code fix if I'm wrong as I'm not familiar with the internals of Roundcube. |
I think your PR solves this issue as well, but I have zero time to invest in this currently. I'll check this later maybe. |
I'm closing this due to inactivity. @bohwaz If you're interested, please check if the latest code on the "master" branch still has your issue and maybe re-open if appropriate. |
@bohwaz: Can you check your PR? |
Don't have time right now, sorry. Feel free to do so :) |
This doesn't add more CPU usage as it is using the already called HTML filtering function to discover if an inline part is referenced or not.
This fixes issue #5051 and is a bit better than #9150