-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Consider velocity while EndDragging. #53
Comments
Hello @ImZarK , thanks you for raising this issue! I completely re-developed the library using a custom collection view layout and I tried my best to implement your logic with the velocity/scroll threshold. Please take a look and let me know if it behaves as expected. If that's the case feel free to close this issue 👍 |
@MaherKSantina Good morning Maher! thanks for your library it's amazing, but in this new version I'm having the following issue http://www.giphy.com/gifs/j45AZB2dGj3FGJvFJH |
Hello @Sk8er22 ! Thank you for your feedback really appreciated! And thank you for raising this issue. I'm not able to see the gif, I'm getting this error: |
Ohh that is very bad... I'm sorry you're experiencing this, so let's see how we can fix it. I can't investigate this from the gif as I don't know what's causing this. Are you able to copy the content of the collection view in the example project and see if the issue is still there? This way we can isolate the issue and work on it |
And the vc:
I'm doing somthing wrong? Thanks for your fast answer 👍 |
Hello @Sk8er22 , it looks like you're configuring the behavior properly. I can't see any issue from the code you wrote. I'm assuming it's a UI issue where the each cell frame is not computed properly. Are you using a library for the cell items UI? I noticed that when you long press one of them it's becoming smaller, or is it just the selection style? |
Just right now I was trying it commenting the highlighting behavior... but the error persists...
|
Hmm... so the highlighting is not the issue, and you're not using an external library. Can I bother you to import the collection view code and item UI to the example app and test there? It would be very easy for me to debug the issue if it's in the example project. You can fork the repo and push the UI code and I can pull your branch and investigate it |
@Sk8er22 thank you soo much for sponsoring me!! 🎉 🎉 🎉 🎉 It really motivates me to pursue my vision and help others with their UI challenges! <3 |
The pod with the Large display card is this one... |
Ohh okayy I'll have a try and will let you know how it goes |
@Sk8er22 I was able to reproduce the issue in the example project! I'll try to investigate and work on a fix now |
@Sk8er22 I've released a new version of the pod that fixes this issue! Thank you so much for helping me investigate this! I also found another issue related to this but I wanted to push an update for you so you don't have to wait for the other fix. I'll create another ticket for the other issue Have a good day! 😄 |
Thank you so much! :) |
Hello @ImZarK , I'm going to close this issue as there has been another thread going on for another issue, but feel free to open a new issue if you noticed that it's not working as intended! Thank you very much for raising this issue and contributing ❤️ |
Act more like isPagingEnabled.
If the velocity of EndDragging is zero, use "if scrollDistance more than half the page width" to decide destinationIndex, instead of "scrollThreshold". #52 #
The text was updated successfully, but these errors were encountered: