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

Remove downloaded scripts and replace with CDN #475

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

Conversation

rahatzamancse
Copy link
Contributor

I think using an existing CDN than downloading a third-party script locally is a better approach. This makes the local repository smaller.

@schneidr
Copy link

schneidr commented Mar 7, 2023

Please leave an option to use the theme without a CDN. CDN are problematic from a privacy point of view, since they allow the CDN to track users across websites.

rahatzamancse and others added 3 commits October 2, 2024 10:54
* delicious and google bookmarks discontinued

* delicious and google bookmarks discontinued

* fix: i18n keys

* Social services i18n keys had "line" as a key, fixed accordingly.

* fix: added missing share services

Added "telegram" and "whatsapp" in the list.

* fix: breadcrumb not turning-off when in term list

When breadcrumb is turned-off in config file, it remains available in `/categories/{term}/` and `/tags/{term}/ ` list view.

* fix: breadcrumb not turning-off when set in config

Fixed breadcrumb showing in "Section List" regardless of "false" setting in config params.

* upd: deprecated in Hugo v0.119.0, removed in Hugo v0.125.0

* fix: use hugo.IsServer as .Site.IsServer is deprecated

---------

Co-authored-by: I'M YourOnly.One <[email protected]>
Co-authored-by: techmagus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants