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

Include libraries via CDN instead of embedding #86

Open
kordwarshuis opened this issue Feb 20, 2025 · 0 comments
Open

Include libraries via CDN instead of embedding #86

kordwarshuis opened this issue Feb 20, 2025 · 0 comments
Assignees
Labels
Study The result is a report of an investigation

Comments

@kordwarshuis
Copy link

  • What: JavaScript libraries are embedded in the index.html at the moment.
  • Why: the index.html is growing (for example after updating mermaid.js library) and gulp compile takes a very long time, also the size of index.html goes over 5 MB if many terms
  • When: after discussing with @henkvancann
  • What way: Remove files like mermaid.js and add a link to CDN in HTML template
  • Where: https://github.com/trustoverip/spec-up-t/tree/master/assets

Libraries:

  • mermaid.js
  • chart.js
  • popper.js
  • tippy.js
  • prism.js
  • notyf.js
  • font-awesome.js

And new to add:

  • bootstrap.css / bootstrap.js

Pro:

  • Faster site
  • Smaller site
  • Easier to maintain

Con:

  • If CDN is down, the site does not work
@kordwarshuis kordwarshuis added the Study The result is a report of an investigation label Feb 20, 2025
@kordwarshuis kordwarshuis self-assigned this Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Study The result is a report of an investigation
Projects
None yet
Development

No branches or pull requests

1 participant