mailgun / transactional-email-templates

Responsive transactional HTML email templates
http://blog.mailgun.com/transactional-html-email-templates/
MIT License
6.9k stars 1.15k forks source link

Capitalize the "M" in "Margin", to force proper margins in Outlook.com #21

Closed lembitk closed 6 years ago

lembitk commented 8 years ago

Closes #18.

idmontie commented 8 years ago

Wouldn't fixing this in the Premailer be better than applying it here?

There is already an issue open for this too: https://github.com/premailer/premailer/issues/276

lembitk commented 8 years ago

Thanks @idmontie, I see your point, but do not agree.

With all that said – thanks for the insightful comment. It sure would be great if Premailer would add an option for preserving capitalization (and other arbitrary/non-standard/unexpected markup).

P.S. I personally think that the best way to resolve this would be at the build process level – e.g. in a Gulpfile accompanying the templates, after running the CSS inliner, re-capitalize the Margin CSS rule names in the final markup.

leemunroe commented 8 years ago

LGTM :shipit: