search

mapbox / mapbox-maps-android

Interactive, thoroughly customizable maps in native Android powered by vector tiles and OpenGL.
https://www.mapbox.com/mobile-maps-sdk
Other
475 stars 132 forks source link

Point.fromLngLat is dangerous #1005

Closed eygraber closed 2 weeks ago

eygraber commented 2 years ago

I'm updating from v9 to v10, and (aside from the other massive amount of changes), the change from LatLng to Point is made very dangerous by the fact that Point.fromLngLat expects the longitude first, while LatLng had the latitude first.

At the very least there should be a Point.fromLatLng

ZiZasaurus commented 2 years ago

@eygraber thank you for your feedback. The use of the lnglat convention derives from the GeoJSON specification. We will consider your request for adding this convenience in a future version of v10.