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

Prevent override of gmap gestureHandling setting #20

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

antonplagemann
Copy link
Contributor

@antonplagemann antonplagemann commented Sep 15, 2024

Fixes #19

This pr adds the smalles possible change to fix #19, allowing to use the google maps gestureHandling setting in the extension configuration object. I've tested the proposed changes with my application.

It also shouldn't break existing behaviour as shown in the following table (3 behaviour changes marked bold):

roam gestureHandling old behaviour new behaviour
true (default) "cooperative" gestureHandling set to "auto" gestureHandling set to "cooperative"
true (default) "greedy" gestureHandling set to "auto" gestureHandling set to "greedy"
true (default) "none" gestureHandling set to "auto" gestureHandling set to "none"
true (default) "auto" gestureHandling set to "auto" gestureHandling set to "auto"
true (default) undefined gestureHandling set to "auto" gestureHandling set to "auto"
false "cooperative" gestureHandling set to "none" gestureHandling set to "none"
false "greedy" gestureHandling set to "none" gestureHandling set to "none"
false "none" gestureHandling set to "none" gestureHandling set to "none"
false "auto" gestureHandling set to "none" gestureHandling set to "none"
false undefined gestureHandling set to "none" gestureHandling set to "none"

Happy to receive your review @plainheart 😊
And btw, thanks for the great extension! 🙌

@plainheart plainheart merged commit acf1a92 into plainheart:master Sep 17, 2024
1 check passed
@plainheart
Copy link
Owner

Thank you! I will release a new version for this change later.

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.

Can't use option { gestureHandling: "cooperative" } with google maps
2 participants