Skip to content

fix: Stop passing excessive data to the item context #383

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 26, 2025

Conversation

MatiPl01
Copy link
Owner

Description

This PR fixes unnecessary re-renders when the item context is used within the sortable item context caused by the excessive data being passed to the context. I mistakenly put children in there, which usually change on each render.

Copy link

vercel bot commented May 26, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
react-native-sortables-docs ⬜️ Ignored (Inspect) Visit Preview May 26, 2025 9:48am

@MatiPl01 MatiPl01 merged commit 0b466ac into main May 26, 2025
5 checks passed
@MatiPl01 MatiPl01 deleted the fix/excessive-data-in-item-context branch May 26, 2025 09:53
MatiPl01 pushed a commit that referenced this pull request May 26, 2025
# [1.7.0](v1.6.0...v1.7.0) (2025-05-26)

### Bug Fixes

* Invalid call to gesture manager when item is no longer available ([#385](#385)) ([5e5e1ca](5e5e1ca))
* Invalid custom handle measurement ([#384](#384)) ([ad03d2b](ad03d2b)), closes [#377](#377)
* onPress not fired when activation delay is low ([#377](#377)) ([07065b5](07065b5)), closes [#375](#375)
* Order change callback invalid keyToIndex and shadow color interpolation ([#380](#380)) ([c0b3c03](c0b3c03))
* Stop passing excessive data to the item context ([#383](#383)) ([0b466ac](0b466ac))

### Features

* Active item dropped callback, more props in drag start callback ([#381](#381)) ([ef6e6cd](ef6e6cd))
* Add keyToIndex and indexToKey to the item context ([#379](#379)) ([9166043](9166043))
* Add more params to the item drop callback ([#382](#382)) ([36fe591](36fe591))
* Add more touch events to the touchable ([#378](#378)) ([c60500f](c60500f))
@MatiPl01
Copy link
Owner Author

🎉 This issue has been resolved in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant