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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃殌 [REQUEST] - add decoding="async" together loading="lazy" #377

Open
maofree opened this issue Jan 26, 2024 · 3 comments
Open

馃殌 [REQUEST] - add decoding="async" together loading="lazy" #377

maofree opened this issue Jan 26, 2024 · 3 comments
Labels

Comments

@maofree
Copy link
Sponsor

maofree commented Jan 26, 2024

Summary

Hi @Oksydan
right now I've found this solution to increase the loading page speed
add decoding="async" with loading="lazy"
look at here
https://fantinel.dev/web-images-modern-formats#lazy-loading-and-async-decoding

what do you think?

I've seen that astro's website use it in its source code
https://astro.build/

I'm going to test it

bye

Feature target required module/theme

theme

Drawbacks

increase the page speed

@maofree
Copy link
Sponsor Author

maofree commented Jan 26, 2024

It's not easy to understand, but I think it earns some points

@Oksydan
Copy link
Owner

Oksydan commented Jan 29, 2024

Hi @maofree,

yes, using decoding="async" isn't a bad idea.
I am adding this to Falcon 4 roadmap.

@Oksydan Oksydan added the 4.0.0 label Jan 29, 2024
@StefanVonRanda
Copy link

Agreed, decoding="async" is a good default, but loading="lazy" should not be used on any image that visible on load.
You could implement some sort of bounding box check to detect images not currently in view on load, but that defeat the purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

3 participants