Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

[android]Polyline click listener #5919

Closed

Conversation

SylvainHocq
Copy link
Contributor

This is a sample evolution in PolylineActivity to show a solution to do an PolylineClickListener system.

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Aug 9, 2016
@tobrun
Copy link
Member

tobrun commented Aug 9, 2016

Hey @SylvainHocq, thank you for reaching out and contributing, we really appreciate the contribution but we prefer handling this kind of functionality in core/c++ instead of the android/java bindings. In core we can rely on geometry objects and using intersection methods oppose doing the calculations manually. This will not only be a more performant solution but will resolve the issue for multiple platforms at once.

@tobrun tobrun closed this Aug 9, 2016
@SylvainHocq
Copy link
Contributor Author

Hello @tobrun, I have try this because i need this functionality.
And it seem to be enougth efficient even if it's code in Java.
The method getProximityPolyline take around 80ms, it's not so bad...

But yes i understand it's better to code this in C++ cause of more efficient and , for portability of all platform, but i'm not very confortable in c++ :-(

So do you have an idea of when this feature will be coded?
Thanks.

@tobrun
Copy link
Member

tobrun commented Aug 10, 2016

Capturing from @1ec5 on the iOS side:

It's on the roadmap for iOS SDK v3.4.0, First we need #5165, then #5502 will be much simpler.

This will normally translate to a similar timeline for Android so 4.2.0 or the release afterwards. I feel your need for this functionality since I created the #3720 a while back. Sorry for not being able to get this functionality in at this point, we aware this is highly requested feature.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants