Remove Header line spaces
Closed this issue · 7 comments
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
@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.
@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
@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.
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.
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.
@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.