XamEffects icon indicating copy to clipboard operation
XamEffects copied to clipboard

Added a new "hold" command to for hold gestures

Open michaelkollmann opened this issue 4 years ago • 4 comments

Hi!

I have added a new command for both iOS and Android called "Hold".

This command gets triggered if the user presses the control and holds it down for 800 milliseconds. It does not require the user to raise his finger. That's the main difference between this and the Long Tap command.

I have already updated the nuspec-version to 1.7.0 and the README.md file as well.

In theory, you should be able to publish the new NuGet package version of this one right away. I'd greatly appreciate having this as a NuGet package so that I can easily integrate it into the project I am working on at the moment.

Cheers Michael

michaelkollmann avatar Jul 16 '20 09:07 michaelkollmann

Hi, thank you. Hold isn't need since now just "long tap" works incorrect :) If you have a free time - please just fix long command instead adding hold.

mrxten avatar Jul 17 '20 07:07 mrxten

So just to confirm, the "long tap" command is supposed to work like the "hold" command does work now in my implementation?

Not sure about changing the behavior of the "long tap" though. Somebody might use it like it is and be very confused on why we change the behavior of an existing command. That's why I have added a new command instead of changing the old one.

But I can also update the "long tap" command to work just the same as my "hold" command does work now.

What do you prefer?

michaelkollmann avatar Jul 20 '20 14:07 michaelkollmann

No, long tap now works incorrectly 100%. It was correct some time ago.

mrxten avatar Jul 25 '20 20:07 mrxten

I have updated the samples so that every command has its own button to try. Also, I tested it on both iOS and Android.

Both seem to be working as expected for me (now that I have found a bug which caused the Hold command to not work properly)

Can you confirm?

michaelkollmann avatar Jul 30 '20 07:07 michaelkollmann