Skip to content

Conversation

@onnimonni
Copy link
Contributor

Description

I asked feedback about my website and my friend shared great point that the default Youtube preview images are quite blurry.

I learned how img srcsets work and added support for high quality alternative webp photos.

I tested this on locally with chrome and it skips completely the default jpg and preloads only the highest available image on desktop.

Notion Test Page ID

0f7d9b7aec8b4d16a064886b245badef

Preview images

This is how it currently looks in production at https://keksi.io
Screenshot 2024-11-11 at 21 44 51
This is how it looks in my localhost:3000
Screenshot 2024-11-11 at 21 44 09

@vercel
Copy link

vercel bot commented Nov 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
react-notion-x ✅ Ready (Inspect) Visit Preview Nov 11, 2024 8:01pm
react-notion-x-minimal-demo ✅ Ready (Inspect) Visit Preview Nov 11, 2024 8:01pm

@transitive-bullshit
Copy link
Member

@onnimonni this looks great!

in the future, please run prettier and make sure tests pass https://github.com/NotionX/react-notion-x/actions/runs/11785318289/job/32826328578?pr=582

I'll fix locally for now.

@transitive-bullshit transitive-bullshit merged commit f875ba8 into NotionX:master Nov 18, 2024
2 of 4 checks passed
@onnimonni
Copy link
Contributor Author

in the future, please run prettier and make sure tests pass https://github.com/NotionX/react-notion-x/actions/runs/11785318289/job/32826328578?pr=582

I'll fix locally for now.

Thanks and sorry about that 🙇 I'll try to check that it would do it automatically on commits in the future 👍

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