Blog post images should lazy load #568
Conversation
Brenosalv
added
the
tech debt
| const parsed = JSON.parse(imgdata); | ||
| return ( | ||
| <GatsbyImage | ||
| className={className} | ||
| style={{ margin: '0 auto' }} | ||
| image={parsed} | ||
| alt={alt || 'Blog Post Image'} | ||
| loading="eager" |
There was a problem hiding this comment.
Default value is lazy.
| const parsed = JSON.parse(imgdata); | ||
| return ( | ||
| <GatsbyImage | ||
| className={className} | ||
| style={{ margin: '0 auto' }} | ||
| image={parsed} | ||
| alt={alt || 'Blog Post Image'} | ||
| loading="eager" | ||
| placeholder="none" |
There was a problem hiding this comment.
This prop placeholder does not exist in StaticImage component.
|
Visit the preview URL for this PR (updated for commit 4371fcb): https://estuary-marketing--pr568-brenosalv-tech-debt-72zpn15z.web.app (expires Sun, 15 Dec 2024 19:05:21 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 76f6b095a0752e5d9c6c890267f9fdc3e392161e |
|
Images are not showing up with lazy loading: They show up with eager loading. |
|
Forgot that I tried this once and ran into the same issue. I did not look into it at all though - so the fix is possibly simple. |
To be honest, I’m not sure how to fix this yet. The lazy loading just works when we refresh the page. |
Changes
@draftbox-co+gatsby-rehype-inline-images+1.0.2.patch.Tests / Screenshots
Prod env:
Dev env:
