Skip to content
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

fix jump effect error when drag item to top of the list #44

Closed
wants to merge 2 commits into from
Closed

fix jump effect error when drag item to top of the list #44

wants to merge 2 commits into from

Conversation

GabrielBrasileiro
Copy link

@GabrielBrasileiro GabrielBrasileiro commented Aug 12, 2024

Fix jump effect error when drag item to top of the list on example component SimpleReorderableLazyColumnScreen.

To reproduce the effect is only need to drag and drop some item to top of the list again.

fix jump effect error when drag item to top of the list on example component SimpleReorderableLazyColumnScreen
- Update of readme to reflect the example with jump fix
@Calvin-LL
Copy link
Owner

Can you show an example where to.index != toIndex or from.index != fromIndex? I think they are just always the same right?

@GabrielBrasileiro
Copy link
Author

GabrielBrasileiro commented Aug 12, 2024

@Calvin-LL it's right. I opened trying correct this issue updating one abstraction inside of list (In other commit). But I see the solution in the thread. I'm closing the PR.

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