Skip to content

Conversation

JeremyLoh
Copy link
Owner

@JeremyLoh JeremyLoh commented May 8, 2025

For <PodcastEpisodeList>, replace the margin-top with padding-top to prevent layout shift in react-virtuoso

  • https://virtuoso.dev/troubleshooting/#list-does-not-scroll-to-the-bottom--items-jump-around
  • List does not scroll to the bottom / items jump around
    • This is the most common setup error. It happens because the DOM elements inside the items (or the items themselves) have margins. Margins are hard to measure, more so if they collapse. Some elements (for example, p, h*, ul) have default margins.
      Fix
    • Use your browser inspector (the computed tab next to styles) and examine the list items. Margins are usually easy to spot. Replace them with padding.

@JeremyLoh JeremyLoh added bug Something isn't working refactor performance Improvements to app performance labels May 8, 2025
Copy link

vercel bot commented May 8, 2025

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

Name Status Preview Comments Updated (UTC)
xtal ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 3:26pm

@JeremyLoh JeremyLoh merged commit 9b6e3a5 into main May 8, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working performance Improvements to app performance refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove CLS due to LoadingDisplay not wrapping all UI elements

1 participant