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.

Sign up for GitHub

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

Jump to bottom

Fix setOnTouchListener limitations #1157

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix setOnTouchListener limitations #1157

wants to merge 2 commits into from

Conversation

Ph0tonic
Copy link
Contributor

@Ph0tonic Ph0tonic commented Sep 15, 2020 •
edited
Loading

Upgrade mapboxMapSdk version. This new version allows to fix #884 and #1140.

\cc @tobrun and @LukasPaczos

@Ph0tonic Ph0tonic marked this pull request as ready for review December 29, 2020 12:57
@Ph0tonic
Copy link
Contributor Author

Ph0tonic commented Dec 29, 2020 •
edited
Loading

As #1158 bumps map sdk to version 9.5, this new version includes a new api which allows to fix #884 and #1140 see mapbox/mapbox-gl-native-android#511.
\cc @LukasPaczos, @kiryldz and @tobrun

@Ph0tonic Ph0tonic mentioned this pull request Dec 29, 2020
Open
@Ph0tonic
Copy link
Contributor Author

Hello,

Any updates on this, this functionnality brings a big improvement as it allows developpers to have setOnTouchListener on the mapView while still using the annotation plugins. (Solving #884 and #1140)

Let me know if I can do anything else,

@mapbox/maps-android

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

mapView.setOnTouchListener#onTouch is not invoked with Annotations
1 participant
@Ph0tonic