Skip to content

Add lazy loading attribute #79

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

Closed
wants to merge 1 commit into from
Closed

Add lazy loading attribute #79

wants to merge 1 commit into from

Conversation

SUPERCILEX
Copy link

Fixes #78

@DirtyF
Copy link
Member

DirtyF commented May 16, 2020

From what I understand the loading attribute is meant for images and iframes, not scripts.

What you want here is maybe something like defer or async? https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script

@SUPERCILEX
Copy link
Author

Oops! Yeah, I thought I'd read that it worked with scripts, but I just checked and the attribute is ignored. I also tried async and defer, but both of those break rendering for some reason.

@SUPERCILEX SUPERCILEX closed this May 16, 2020
@jekyll jekyll locked and limited conversation to collaborators May 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support lazy loading attribute
3 participants