search

danschultzer / premailex

Preflight for your HTML emails - inline styling and plain text.
MIT License
172 stars 20 forks source link

Remove inline css from markup #14

Closed willbarrett closed 6 years ago

willbarrett commented 6 years ago

Includes a commit from https://github.com/danschultzer/premailex/pull/13, so this PR will get smaller once that one is merged.

Inspired by code in https://github.com/danschultzer/premailex/pull/9/files, updated to support multiple parsers.

@danschultzer

willbarrett commented 6 years ago

Thanks for the reviews and the merge last night @danschultzer - I've merged in master and this should be ready for another look. Using the meeseeks library retains the doctype, so this should be the end of the changes we need for now. Thanks a ton for working on this with me!

danschultzer commented 6 years ago

Looks good @willbarrett, and sure no problem. Thanks for your help improving the library!

I've some thoughts on this PR but got some unrelated work to take care of beforehand. I'll get back to it ASAP.

willbarrett commented 6 years ago

Sounds good. I'll be around!

danschultzer commented 6 years ago

I'll close this in favor of #18