perf: don't update on scroll/rect changes unless virtualized range changes

[mogelbrod]

Fixes #46

• Avoid state updates caused by useScroll() that doesn't cause the virtualized range to change
• Remove the unnecessary reversedMeasurements array and scrollOffsetPlusOuterSize variable
• Remove the no longer used useScroll hook

[tannerlinsley]

Looks good! Can you rebase with the last 2 PRs that came in?

[mogelbrod]

Looks good! Can you rebase with the last 2 PRs that came in?

Done!

I also removed the useScroll hook that I couldn't get to work in favour of some (hopefully) more streamlined code.

[uonr]

list doesn't corrently rendering when the parent containter wasn't mounted in the beginning.

[mogelbrod]

@tannerlinsley This PR should be ready for another pass now 👍

[tannerlinsley]

Looks good!

[tannerlinsley]

🎉 This PR is included in version 2.2.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[gausie]

@mogelbrod @tannerlinsley I think this change caused #68 and #56 because it attempts to interact with the parentRef element without checking whether it has been assigned (e.g. https://github.com/tannerlinsley/react-virtual/blob/master/src/index.js#L83)

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 0cb45f0.

Command	Status	Duration	Result
nx affected --targets=test:sherif,test:knip,tes...	✅ Succeeded	<1s	View ↗
nx run-many --target=build --exclude=examples/**	✅ Succeeded	<1s	View ↗

---

☁️ Nx Cloud last updated this comment at 2025-06-01 10:33:40 UTC