-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
GeastureRecognizers are not updated via UpdateIncremental on iOS #11
Comments
@Dolfik1 Hmm strange. Fabulous use the exact same logic for all the platforms, so It is more likely to be a bug in Xamarin.Forms (well, at least on the iOS implementation) which seems to be a common occurrence according to Google. Or if you wrote your own CollectionView renderer, it could be an invalid use of Collections can be tricky to handle because of virtualization, the cells are reused and you need to make sure to keep the correct ViewElements that were used to generate those cells. Otherwise you can end up with mismatch. Fabulous only compares "previous" and "new" ViewElements to decide what to update in the cell. |
@TimLariviere I am also experiencing this issue. Taps defined in CollectionView cells fire multiple times after the cell has been recycled. It looks very much as if the event handler for the GestureRecognizer isn't being unhooked when the BindingContext of the cell changes. I seem to recall a fix for something like that in the latest XF 5.0 release. If that's a Xamarin Forms bug, would Fabulous 0.60 be able to work with Xamarin Forms 5.0? |
@programmation Thanks for your report. Would you happen to have a reproduction that you can share? Regarding XF 5.0, it's not currently possible to use it as there are a bunch of breaking changes. fabulous-dev/Fabulous#836 is aiming to support it. |
@TimLariviere Thanks for the information about XF 5.0. I'll hold off upgrading until it is supported. I'm also trying to put a sample project together to demonstrate the Taps problem in CollectionView cells. |
We have out implementation of CollectionView that frequently uses
UpdateIncremental
function. I find out that sometimesgeastureRecognizers
are not updated. There is examle:When I run this example I see different values in UI and stdout. This reproduced only on iOS.
The text was updated successfully, but these errors were encountered: