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

Templates: use relative URLs for static assets #505

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

@lt3Dev
Copy link

@lt3Dev lt3Dev commented Aug 21, 2021

I noticed most asset links use relative URLs, except for a few. This commit remedies this inconsistency. OpenGraph embeds were not changed, because in my experience, some embed scrapers require absolute URLs.

Motivation: The site I use has an onion and a clearnet site, and currently, visiting the onion loads fonts, styles, and the favicon
over the clearnet.


  • I have signed the CLA
@lt3Dev lt3Dev force-pushed the pr/relative-urls branch from 638c267 to 5184dee Aug 21, 2021
I noticed most asset links use relative URLs, except for a few. This commit remedies this inconsistency.
OpenGraph embeds were not changed, because in my experience, some embed scrapers require absolute URLs.

Motivation: The site I use has an onion and a clearnet version, and currently, visiting the onion loads fonts, styles, and the favicon
over the clearnet.
@lt3Dev lt3Dev force-pushed the pr/relative-urls branch from 5184dee to cc69f9f Aug 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant