jonkemp / gulp-inline-css

Inline linked css in an html file. Useful for emails.
MIT License
272 stars 29 forks source link

Ignoring style tags or link tags on inline #15

Closed jpsc closed 1 year ago

jpsc commented 9 years ago

I would be nice to have the possibility to bypass any style or link on the inline.

Something like this:

<!-- external styles -->
<link rel="stylesheet" data-ignore="ignore"  href="style.css" />

<!-- embedded styles -->
<style data-ignore="ignore"></style>

Idea from here.

What do you think ?

jonkemp commented 9 years ago

Would any of the juice options achieve what you want?

jpsc commented 9 years ago

I don't think so. With those options you can add all or none. My idea is to just keep specific ones from going inline. As well all know there are styles that don't need / can't be inlined on html newsletters so could be nice to have that option.

The preserve media queries option comes close to that, at least with the final output, but can't put those styles in a media query because they won't be read by some email clients.

jonkemp commented 9 years ago

Thinking about this I'm not sure how it's possible. Basically all this module does is make the juice(2) module work with gulp. So it's really just a wrapper around that module. So if juice doesn't support it, then I don't see how it will work.

jpsc commented 9 years ago

Yes. I've seen that now. I'll go directly to juice :pray:

jonkemp commented 9 years ago

This plugin uses a new module called inline-css, which is it's own version of juice. So the feature request needs to be made here or in the inline-css repo.

jonkemp commented 9 years ago

@kaiserkoenig I think you left this comment on the wrong issue. The templating issue is here #4.

kaiserkoenig commented 9 years ago

ups... monday morning

jpsc commented 1 year ago

No longer relevant