Remove Header line spaces

[anoreg]

It's really a nice plugin after smarttemplate4 become a commercial plugin. But there's a tiny issue: My outlook reply & forward header doesn't have line space It looks like:

From: name Sent: xxx ... Subject: xxx Msg body ...

I tried using replay or forward and send msg to myself The format looks good in thunderbird but if received mail in outlook, the header shows:

From: name "a empty line " Sent:xxx ... Subject: xxx "a tab intent" Msg body

ThunderBird version60.8.0 (32bit) RWH version2.1.0

[jeevatkm]

@anoreg Thanks for reporting an issue. I will have to analysis and fix the problem. These sort of issues are typically hard to narrow down the root cause.

[stepkas]

@jeevatkm Hello, thanks a lot for the addon! I'm also facing an issue with line spacing when reading the mail in outlook v9 and above. From what i understand it uses Word for rendering HTML and it adds <p> tags to the header made with the addon.. That makes outlook users see an extra space between header lines..

Attaching how the header looks in Thunderbird (html_header.txt) and how outlook sees it (outlook_header.txt).

When/if you have time please check if it's possible to fix this. This might help: https://stackoverflow.com/questions/18379511/outlook-com-put-extra-p-around-my-spaces

Thanks a lot in advance and keep up the great work! outlook_header.txt html_header.txt

[jeevatkm]

@stepkas Thanks for reporting an issue. Actually RWH does not insert/add any <p> tag and mso-* stylesheet, however your attached outlook_header.txt has those <p> and mso-* stylesheet. It seems outlook is added it.

I have read your stack overflow reference. Suggestion is use table, tr and td. I will look into it.

[jeche17]

I'm also having the same issue with line spacing. Want to know if this problem solved yet or not? How to resolve this problem? Thanks.

[cnmicha]

Same issue here, RWH seems to use <div>...</div> tags for the header lines. This causes Outlook to insert a vertical spacing. I think we could fix this by replacing those tags by <span>...</span><br>. I changed the addon code locally and can confirm this fixes the issue.

[cnmicha]

@jeevatkm See https://github.com/jeevatkm/ReplyWithHeaderMozilla/blob/master/chrome/content/core.js#L366-L368. Here we would need to do the change. If you like, I can provide a pull request.

[jeevatkm]

@anoreg @stepkas @jeche17 @cnmicha Please see https://github.com/jeevatkm/ReplyWithHeaderMozilla/issues/103#issuecomment-2106526608