Feedback about mapbox-maps-flutter HOT 4 OPEN

If we stick to using GeoJSON, won't we always have this problem? The 2 points mentioned in #20 (comment) can be ignored if we just drop the GeoJSON requirement.

I think users would have a nicer time if they use types rather than need to create untyped JSONs. Why don't we use types provided by library?

from mapbox-maps-flutter.

MapboxView & MapboxController would be more idiomatic

from mapbox-maps-flutter.

Hi @ben-xD,

you actually don't have to use turf here. The underlying data standard is GeoJSON. GeoJSON is itself based on JSON to represent geo data.
And if you want GeoJSON in a serialized Dart-representation, it consists of Maps, Lists, and numbers.

So the GeoJSON that you want to inject into the CameraOptions.center attribute can look like this (without turf library):

CameraOptions(
  center: {
    "type": "Point",
    "coordinates": [-80.1263, 25.7845]
  }
  // ...
),
// ...

Or isolated:

{
    "type": "Point",
    "coordinates": [-80.1263, 25.7845]
}

What turf simply provides, is an optional to use class-object representation of GeoJSON. Check it out in the README of the turf library https://github.com/dartclub/turf_dart.

The output of the Point.toJson method is the same Dart Map/List/number representation as shown in my example.

The benefits of using turf with Mapbox can be static analysis and runtime checks, to make sure if the GeoJSON data structures are valid.

from mapbox-maps-flutter.

LatLng center;

A new data type LatLng could add a lot of confusion, because:

• It does not really correspond to the Point or Position/coordinate data type from the GeoJSON RFC standard, that Mapbox is using for the representation of the geo data.
• The actual order of the Position's/coordinate's dimensions are: [LONGITUDE, LATITUDE, ALTITUDE (optional)] See here RFC. So to stay compliant with the GeoJSON standard regarding the order of the dimensions is important.

Though I totally see, that it would make sense to have more type information than just Map<String?, Object?> ("a Map with some String? keys and some Object? values")...

from mapbox-maps-flutter.