I have the following e-mail template:
Which in turn creates the following e-mail
Content-Type: text/html; charset=UTF-8
<p><div style='margin-left: 15px'><br>[text]</p>
<p><a href="[link]">Visit Topic</a> or reply to this email to respond.</p>
<p>You are receiving this because you enabled mailing list mode.</p>
<p>To unsubscribe from these emails, <a href="[LINK]">click here</a>.</p>
<div style='margin-left: 15px'> creates an unnecessary whitespace indentation and seems odd in itself.
I assume you are objecting to the suppression of avatar in single short email notifications? This was done to satisfy Spam Assassin rules where small emails with images are penalized.
No, I disabled the header because a user asked for it and we wanted to make the mailing-list experience more mailinglist like
My issue is that the first paragraph is indented and that makes it look like it might be a citation (which it is not). Looking at the raw html of the e-mail there is a spurious
<div style='margin-left: 15px'>.
Oh I see @neil can you have a look?
Yeah I see the problem.
result.gsub!(/\[email-indent\]/, "<div style='margin-left: 15px'>")
That will create some mangled html.
Looks like there was old code trying to indent posts, and it probably worked great… for two days in 2013. This commit became obsolete two days later by this commit when a custom html template was used instead of the markdown. Now we allow customized email templates to be used for HTML, which unearthed this archaeological treasure.