Optionally ignore style tags in HTML, accept additional argument for css to inline? about css-inline HOT 4 CLOSED

those look awesome!!! please feel free to close this issue :)

from css-inline.

Hi @cheapsteak !

I didn't think about it, but having such a feature seems perfectly reasonable to me :) Correct me if I am wrong, but to support your use case, it might be:

• A InlineOptions option to disable "style" tag inlining;
• A InlineOptions option with extra CSS to inline;

As far as I see, the extra CSS should be in the form of qualified rules as a single string, e.g. h1 {color: green}, is it in this way in mjml?
Also, from looking at the code sample you mentioned I am not sure what insertPreservedExtraCss option does? Could you, please, help me with that?

If the things above make sense, then I can implement that :) And I will appreciate if you can provide some input, expected output samples so I can include them to the test suite :)

Cheers

from css-inline.

A InlineOptions option to disable "style" tag inlining;
A InlineOptions option with extra CSS to inline;

yes that should work!

the extra CSS should be in the form of qualified rules as a single string, e.g. h1 {color: green}, is it in this way in mjml?

yup!

insertPreservedExtraCss

that's a "juice" parameter , i'll copy their description here :)

whether to insert into the document any preserved @media or @font-face content from extraCss when using preserveMediaQueries, preserveFontFaces or preserveKeyFrames. When true order of preference to append the <style> element is into head, then body, then at the end of the document. When a string the value is treated as a CSS/jQuery/cheerio selector, and when found, the <style> tag will be appended to the end of the first match. Defaults to true.

for emails, extra CSS passed in separately from the html that can't be inlined should probably be thrown away, which I guess is why mjml had disabled that :)

from css-inline.

Thanks! I implemented the abovementioned options - inline_style_tags & extra_css (in #46 and #48 ). Is there anything else I can do regarding this feature?

I think that a new release could be issued when #49 is done so you can directly use it as a replacement for juice :) What do you think?

from css-inline.