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 issue #101 #123

Closed
wants to merge 1 commit into from
Closed

Fix issue #101 #123

wants to merge 1 commit into from

Conversation

grandsir
Copy link

this commit adds a modifier called enableAnimationsOnDragIndicator, which enables the animations on drag indicator, it is false by default, so that issue on 101 is fixed by default as well, only will occur if you add enableAnimationsOnDragIndicator to the view, I added that because maybe someone will want to add some animations, more customization is better.

the left phone is with enableAnimationsOnDragIndicator , and the right is without it.

Screen.Recording.2023-03-11.at.13.42.52.mp4

@lucaszischka
Copy link
Owner

Hello @grandsir,

sorry for my very late reply. I think #101 is a bug, not something we should keep as a feature that can be enabled. Or are there any side effect to disabling the animations on the drag indicator? Therefore I would happy to merge this pull request, if you update it! Also please include a Changelog mention.

Greetings from Passau,
Lucas

lucaszischka added a commit that referenced this pull request Jun 26, 2023
@lucaszischka
Copy link
Owner

Hello @grandsir,
because you have not replied since my last message, I simply fixed it myself instead of merging the pull request (see commit 8a378a8). However I mentioned you in the release notes!
Thanks, Lucas :)

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