Skip to content
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

Adding MF Swipe recognizer and Trigger zone recognizer #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darthbator
Copy link
Contributor

These are two new recognizers that I have written. One that tracks a single touch starting in a boundary until it ends and the other a version of the swipe recognizer that is able to simultaneously track swipes across multiple fingers.

@prime31
Copy link
Owner

prime31 commented Jun 28, 2017

Looks good to me. If you can just swap it over to the coding style the rest of TouchKit uses (there is an mdpolicy file in the root of the repo with the formatting baked in) I can get this merged in.

@darthbator
Copy link
Contributor Author

I went ahead and imported the policy into monodev, made sure it was being used in the options and then used Edit>Format>Format Document but didn't see any format changes occur. The selected policy is NewP31XamarinPolicy. I'm not ultra familiar with formatting documents in Monodev so any help would be appreciated. I am assuming I'm doing something wrong?

@prime31
Copy link
Owner

prime31 commented Jun 29, 2017

Xamarin has 3 levels for its formatting (and all of its settings for that matter):

  • default policy (which is probably what you set when you pulled in the policy file)
  • solution (right click and choose options)
  • project (right click and choose options)

You have to make sure they are all set correctly for Xamarin to pick up on a top level policy. Solution and project must be set to inherited.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants