Skip to content

fix: Invalid custom handle measurement #384

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 3 commits into from
May 26, 2025

Conversation

MatiPl01
Copy link
Owner

Description

This PR fixes issues caused by recent changes introduced in the #377 PR.

I modified the gesture in the custom handle component by adding the onBegin method to the gesture. This didn't work when the custom handle was rendered after the item (e.g. with some timeout). This PR fixes this invalid behavior and simplifies handle measurement logic a bit.

@MatiPl01 MatiPl01 self-assigned this May 26, 2025
@MatiPl01 MatiPl01 added the fix label May 26, 2025
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 3:31pm

@MatiPl01 MatiPl01 merged commit ad03d2b into main May 26, 2025
5 checks passed
@MatiPl01 MatiPl01 deleted the fix/invalid-custom-handle-measurement branch May 26, 2025 15:59
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