Skip to content

chore: support progressive image loading with local assets#1820

Closed
agg23 wants to merge 5 commits intomicrosoft:mainfrom
agg23:progressive-image
Closed

chore: support progressive image loading with local assets#1820
agg23 wants to merge 5 commits intomicrosoft:mainfrom
agg23:progressive-image

Conversation

@agg23
Copy link
Contributor

@agg23 agg23 commented Jul 31, 2025

A unfortunately large diff.

Integrate plugin-ideal-image. We found the implementation automatically detects slow connections and requires a click to load the image. This requires a custom ProgressiveImage component in order to disable the slow connection functionality.

Images will now display an inline, very low res placeholder of the correct size. Images will be progressively loaded. They are also preprocessed with lossy compression to decrease total wire size and physical pixel size.

Screenshot 2025-07-31 at 12 12 56 PM

@agg23 agg23 requested a review from Skn0tt July 31, 2025 19:18
@agg23 agg23 marked this pull request as draft July 31, 2025 19:43
Copy link
Member

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great, thank you! I think CI is failing becuase the plugin isn't registered in docusaurus.config.ts.

@mxschmitt
Copy link
Contributor

Would be great if you could make this PR against the upstream and not against your fork, then we'd have preview environments and could see the impact - thanks!

@Skn0tt
Copy link
Member

Skn0tt commented Aug 1, 2025

Oh, didn't know that the branch source makes a difference! good point.

@agg23
Copy link
Contributor Author

agg23 commented Aug 1, 2025

Closed in favor of #1823.

@agg23 agg23 closed this Aug 1, 2025
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.

3 participants