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

Add VT options to GeoJsonSource #6237

Closed
boundsj opened this issue Sep 1, 2016 · 1 comment
Closed

Add VT options to GeoJsonSource #6237

boundsj opened this issue Sep 1, 2016 · 1 comment
Assignees
Labels
Android Mapbox Maps SDK for Android

Comments

@boundsj
Copy link
Contributor

boundsj commented Sep 1, 2016

In #6217 we are adding support for clustering and VT options to MGLGeoJSONSource (the iOS SDK's version of the Android SDK's GeoJsonSource). While working on that, I noticed that GeoJsonSource does not appear to support VT options available in core. Defaults are provided in core but an Android developer has no way to provide their own.

In addition, GeoJsonSource does not seem to currently provide documentation for the clustering related options it does support. In https://github.com/mapbox/mapbox-gl-native/blob/249514389a77cdb15eee779a332876eea06cdcfa/platform/darwin/src/MGLGeoJSONSource.h the iOS SDK gained some documentation that mirrors the style spec and the Android SDK should probably have at least that, too. However, note that a discussion about how to improve the documentation for a platform is happening in #6236

@ivovandongen @cammace @zugaldia

@boundsj boundsj added the Android Mapbox Maps SDK for Android label Sep 1, 2016
@ivovandongen
Copy link
Contributor

ivovandongen commented Sep 5, 2016

@boundsj I'm actually working on this in #5906, but got blocked on #5701 (probably). Will pick this back up once that lands in master.

See GeoJsonOptions

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

No branches or pull requests

2 participants