-
Notifications
You must be signed in to change notification settings - Fork 5
More generic inliner for other asset types (css, js, img): collaboration? #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hi! Thanks for letting me know about your plugin. We have a few small projects where I work that could definitely use your plugin to create single-page HTML files. I'll let my coworkers know about it. It looks like your plugin handles .svg images by simply replacing the Since your plugin isn't focused on inlining rather than optimization, I think a better approach might be to convert all I found a few existing projects that use the data URI technique, but none for Rehype: |
@JamesMessinger thanks for the detailed input! Based on your suggestion I've already made inlined base64 svgs the default. I think I'm mostly done working on my generic inliner for now. But if I end up extending it some more I may rebase it on your existing inliner. But for now I suggest closing this issue again and I'll give you a heads up if there's any news on my side. Thanks again, |
Happy to help, @marko-knoebl ! |
I've recently created a tiny generic rehype inliner that handles svg, images, css and js:
https://github.com/marko-knoebl/rehype-inline
I'm not sure if any collaboration really makes that much sense as these two are either small or tiny projects. But I just wanted to share that the other project exists and maybe there would be some interest. The main motivation for my project was to create single HTML files that can be downloaded (so network efficiency was not really a concern I wanted to solve).
The text was updated successfully, but these errors were encountered: