-
Notifications
You must be signed in to change notification settings - Fork 48
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
Surely HammerOptions should be __hammerConfig? #35
Comments
I'm also find that, but Should we use ref get DOM and set |
Related to #29 - in case this would be fixed, is there any way to actually set |
I need more detailed information to use config. |
I ended up importing hammerjs directly to configure global options, see https://github.com/ptandler/enneagram-explorer/blob/master/src/main.ts |
I have searched the entire repository and am unable to find any reference to
hammerOptions
. Surely this is meant to be__hammerConfig
?See the following line:
const localOptions = el.hammerOptions && el.hammerOptions[recognizerType];
Should be:
const localOptions = el.__hammerConfig && el.__hammerConfig[recognizerType];
Otherwise
localOptions
is redundant is it not?The text was updated successfully, but these errors were encountered: