[WinDX] Support TouchPanel.MouseTouchPoint/MouseGestures #1304

Merged
merged 1 commit into from Feb 13, 2013

Conversation

Projects
None yet
3 participants
Contributor

mattzink commented Feb 6, 2013

For Windows desktop apps, sends pertinent touch/move/release touch events from mouse events. This gives equivalent behavior that Windows 8 apps have already.

Member

mgbot commented Feb 6, 2013

Can one of the admins verify this patch?

Owner

tomspilman commented Feb 6, 2013

This feature already exists. Just do...

TouchPanel.EnableMouseTouchPoint = true;

Going to close this pull request.

tomspilman closed this Feb 6, 2013

tomspilman reopened this Feb 6, 2013

Member

mgbot commented Feb 6, 2013

Can one of the admins verify this patch?

Owner

tomspilman commented Feb 6, 2013

I didn't read this correctly.

What you are doing here is actually just passing mouse state into TouchPanel. The TouchPanel.EnableMouseTouchPoint still controls if that mouse event is transformed into a touch event or not.

This code correctly does not check for TouchPanel.EnableMouseTouchPoint itself. TouchPanel.AddEvent() needs to get all mouse events so that it can properly manage pending release events even after TouchPanel.EnableMouseTouchPoint is disabled.

I'm going to test this a bit first, but it should be good to merge.

Owner

tomspilman commented Feb 6, 2013

@mgbot - test

Contributor

mattzink commented Feb 7, 2013

@tomspilman Exactly. Sorry if my wording confused you.

Owner

tomspilman commented Feb 7, 2013

@mgbot test

Owner

tomspilman commented Feb 7, 2013

@mgbot retest

Owner

tomspilman commented Feb 7, 2013

@dellis1972 - Mgbot seems to not want to rebuild this.

Owner

tomspilman commented Feb 8, 2013

@mgbot test

tomspilman merged commit 27bc1a2 into MonoGame:develop3d Feb 13, 2013

1 check passed

default Merged build finished.
Details
Owner

tomspilman commented Feb 13, 2013

Thanks... merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment