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

resolves #3961 add loading attribute to image macro #4348

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

Conversation

KartikSoneji
Copy link

@KartikSoneji KartikSoneji commented Aug 30, 2022

Adds image::[loading=eager | loading=lazy] attributes to the image macro.
Also adds global image-loading attribute that can used to set the loading policy document-wide.

resolves: #3961

@KartikSoneji
Copy link
Author

Hi @mojavelinux is there anything pending for this pull request?
I'm not sure where the docs for this would go, the docs folder doesn't seem to have a section on the image macro.
Please let me know what the next steps should be.

@mojavelinux
Copy link
Member

I'm extremely busy right now and won't be able to look at least until next week. But I'll do my best to prioritize it so we can get it wrapped up.

@KartikSoneji
Copy link
Author

Appreciate you looking into it.

@slonopotamus
Copy link
Contributor

@mojavelinux Maybe busy week has ended?)

@mojavelinux
Copy link
Member

My busy week never ends. I'm currently focused on the specification. Once that's well in motion, only then will I get back to moving this project forward again towards 2.1.x and beyond.

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.

Add lazy load attribute for images
3 participants