jeevatkm/ReplyWithHeaderMozilla

Replace div with span

rocky7x opened this issue · 3 comments

Hi,

Just a quick suggestion: could you please replace the HTML prefix and suffix div with span, since when sending the mail to Outlook, it puts space between the lines, so they look as separate paragraphs?

Thank you very much.
Cheers

I did some corrections to the core.js file myself and got a bit better formatting of the header, which mimics the original Outlook header much better. I attach the complete XPI file here (as ZIP, just rename it to XPI), it has add-on version 2.6.3. You can compare my file with yours and if you like it, you can adopt the changes. What I did:

  1. replace hr horizontal line tag with div with border-top (this is the way Outlook does it)
  2. replace all other divs with spans, in order to avoid Outlook putting white space between lines
  3. corrected the margin in the div at the top, using padding instead of margin (this is the way Outlook does it)
  4. explicitly added 0 margin into the htmlTagPrefix variable to prevent the white-spacing done by Outlook parsing

Cheers..
replywithheader-2.6.3-tb.zip

@rocky7x Thanks for your contribution. I'm sorry for the delayed response. I'm starting to work on webextension of the Add-on, I will incorporate your inputs. Thanks.