Skip to content

Update recommendation for lazy loading images beneath the fold#404

Open
kevinfarrugia wants to merge 2 commits intoYellowLabTools:masterfrom
kevinfarrugia:lazy-loadable-images-message
Open

Update recommendation for lazy loading images beneath the fold#404
kevinfarrugia wants to merge 2 commits intoYellowLabTools:masterfrom
kevinfarrugia:lazy-loadable-images-message

Conversation

@kevinfarrugia
Copy link

I think we should recommend native image lazy loading ahead of JavaScript implementations.

@kevinfarrugia kevinfarrugia marked this pull request as draft February 5, 2024 12:10
@kevinfarrugia kevinfarrugia marked this pull request as ready for review February 6, 2024 11:57
@kevinfarrugia
Copy link
Author

I was finally able to run this locally. Seems to work as expected.

image

@Jieiku
Copy link

Jieiku commented Feb 6, 2024

I would change the wording, since the recommendation is coming from a tool.

so instead of:

I recommend using loading="lazy"

maybe:

Using loading="lazy" is recommended.

This may not be perfect wording either, but it avoids the use of "I".

@kevinfarrugia
Copy link
Author

SGTM. Will wait for @gmetais 's feedback since I tried to keep it consistent with the current wording.

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.

2 participants