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

Gesture fixes #851

Merged
merged 5 commits into from
Jan 13, 2022
Merged

Gesture fixes #851

merged 5 commits into from
Jan 13, 2022

Conversation

felix-ht
Copy link
Collaborator

Fixed #777 by treating all gestures as a group on web. This PR supersedes #790

The reason that this is required is that some gestures share setting with others on web - and cannot be changed independently.

I also added a new variable to disable the double click to zoom gesture on web.

While testing double clicks i noticed and fixed that queryRenderedFeatures was broken for web (Wrong object type used with the gl js lib).

I also did some minor/bugfixes on the User interface widget in the example, that i also noticed to be broken while testing the changes.

* rotate now works as expected
* added option to disable double clicks
* fixed queryRenderedFeatures on web
@felix-ht felix-ht added the web label Jan 12, 2022
@felix-ht felix-ht had a problem deploying to ANDROID_CI_DOWNLOADS_TOKEN January 12, 2022 17:49 Failure
@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 12, 2022 17:49 Inactive
@felix-ht felix-ht requested a review from m0nac0 January 12, 2022 17:50
@felix-ht
Copy link
Collaborator Author

@metafounder it would be great if you could take a look as well

@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 10:18 Inactive
@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 10:18 Inactive
@tobrun tobrun temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 15:56 Inactive
@tobrun tobrun temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 15:56 Inactive
@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 17:46 Inactive
@felix-ht felix-ht temporarily deployed to ANDROID_CI_DOWNLOADS_TOKEN January 13, 2022 17:46 Inactive
@felix-ht felix-ht merged commit d3954e5 into master Jan 13, 2022
@felix-ht felix-ht deleted the gesture-updates branch January 13, 2022 17:53
@felix-ht felix-ht mentioned this pull request Jan 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rotateGesturesEnabled seems to be ignored for web
2 participants