Skip to content
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

Embed external resources #90

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ArmanMazdaee
Copy link
Member

What this PR does / why we need it:
Add support for embedding external resources in the rendered HTML so theme developers could place their styles and other external resources in external files

@stp-ip
Copy link
Member

stp-ip commented Mar 24, 2019

Overall it looks good. Should finish the full review tomorrow.

Some notes already that most likely should be morphed into issues, if you agree to handle longer term:

Make embedding default, but able to switch it off? That would allow someone to easily host it as a website and also make it easier for devs. By default the single output file is great for users and later for us to transform it into a PDF or?

Should we cache images? Say by default anything that was fetched should be cached locally and used in subsequent renders. Question is when to invalidate? Only on url change? On url change and a flag to disable the cache?

Local file support instead of a full url?
Such as /image.png, which looks for a image.png file within the current directory?
That would make it easy to manually store image files etc.

Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants