1

Closed

HoldGesture engine and trigger [patch]

description

I've created a hold gesture trigger with a corresponding engine to handle hold gestures. I don't know what the correct procedure is in the windows world for distributing patches, but being an pre linux/open source guy I figured I'd just attach the patch file :)
 
API-wise it's just like the tap gesture engine/trigger. It's not extremely well tested but it worked fine when I tried it in the view in Blake.NUI.TestApplication.Common
 
The patch is against latest subversion checkout, let me know if you need it in some other format than unified diff.

file attachments

Closed Oct 7, 2010 at 6:34 AM by JoshB
Fixed with Patch #6945 by isaks, commited in changeset #67526

comments

JoshB wrote Sep 22, 2010 at 4:46 PM

Since TagGestureTrigger already includes a long press mode, along with other customizations, I'm inclined to reject this patch. Please don't take this as an indication that I don't want patches though, just in this case there was a misunderstanding in the included functionality. In the future please do submit patches. You can upload them on the Source Code tab in the "upload patch" link.

wrote Sep 22, 2010 at 4:47 PM

isaks wrote Sep 22, 2010 at 7:19 PM

Nice. If the tap trigger already does this (press-hold-don't-release) then I'm totally ok with not accepting the patch (but we could still discuss whether this functionality should be bundled in the TapGestureTrigger or if it belongs on its own. It's IMHO two different gestures with two different, well defined names)

And I won't take it personally if it's rejected :-)

JoshB wrote Sep 22, 2010 at 7:43 PM

Functionality already exists in TagGestureTrigger



** Closed by JoshB 9/22/2010 9:47 AM

JoshB wrote Sep 22, 2010 at 7:43 PM

Misunderstood the requested behavior - re-opening.

wrote Sep 22, 2010 at 7:43 PM

wrote Oct 7, 2010 at 6:33 AM

wrote Oct 7, 2010 at 6:34 AM

wrote Feb 14, 2013 at 5:57 PM

wrote May 16, 2013 at 8:31 AM