mapbox / mapbox-maps-flutter

Interactive, thoroughly customizable maps for Flutter powered by Mapbox Maps SDK
https://www.mapbox.com/mobile-maps-sdk
Other
245 stars 93 forks source link

Point Annotation image update not working on ios #532

Open thetranquila opened 2 weeks ago

thetranquila commented 2 weeks ago

Even though it is working on Android. Updating point geometry in ios also works properly

maios commented 1 week ago

Hi @thetranquila, thank you for raising this bug, I have created an internal ticket to track the work https://mapbox.atlassian.net/browse/MAPSFLT-212

bakaemon commented 6 days ago

Could be related that on real IOS 12 device the annotation in some case disappeared or not updating properly.

thetranquila commented 6 days ago

Tested on Iphone 13 Ios 17 device. Also it never updates

maios commented 6 days ago

Hi all, while our team is working on this bug, what you can do to get the image update on iOS is to give a random iconImage each time you want to update image

pointAnnotation.iconImage = "some-other-id";

MapboxMaps iOS has a mechanism to manage images used by PointAnnotationsManager, when the annotations get updated, if there is already an image added to the renderer's style with same id, we will not do anything, so to work around it, you need to give different iconImage (this is gonna be the name/id of the image to add to style)

bakaemon commented 5 days ago

Hi all, while our team is working on this bug, what you can do to get the image update on iOS is to give a random iconImage each time you want to update image

pointAnnotation.iconImage = "some-other-id";

MapboxMaps iOS has a mechanism to manage images used by PointAnnotationsManager, when the annotations get updated, if there is already an image added to the renderer's style with same id, we will not do anything, so to work around it, you need to give different iconImage (this is gonna be the name/id of the image to add to style)

I don't know what I did wrong, but I am getting issue in the following code, using your workaround:

  Future<void> _updateBusAnnotation(
    String busImage,
    Position position, [
    double iconSize = 1,
  ]) async {
    _tmpIcon ??= (await rootBundle.load(busImage)).buffer.asUint8List();
    if (busAnnotation == null) {
      busAnnotation = await pointAnnotationManager?.addAnnotation(
        point: position.toPoint(),
        dataImage: _tmpIcon,
        iconSize: iconSize,
      );
    } else {
      _tmpIcon = (await rootBundle.load(busImage)).buffer.asUint8List();
      if (Platform.isIOS) {
        final String randomString = () {
          const _chars =
              'AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz1234567890';
          final _rnd = Random();
          const length = 10;

          return String.fromCharCodes(Iterable.generate(
            length,
            (_) => _chars.codeUnitAt(_rnd.nextInt(_chars.length)),
          ));
        }();
        busAnnotation = busAnnotation!.copyWith(
          iconImage: randomString,
        );
      }
      busAnnotation = busAnnotation!.copyWith(
        geometry: position.toPoint().toJson(),
        image: _tmpIcon,
      );

      await pointAnnotationManager?.update(busAnnotation!);
    }
  }

However, the icon is guaranteed to disappear if I update the annotation with iconImage assigned instead of randomly disappear. Did I do something wrong?

maios commented 5 days ago

Is this function copyWith something you added from your codebase? If so can we have some snippet code of what it does?

bakaemon commented 4 days ago

@maios

Sorry for the late reply, it was late when I posted the last comment. It was just an extension method I added. Here you go:

 PointAnnotation copyWith({
    Map<String?, Object?>? geometry,
    Uint8List? image,
    IconAnchor? iconAnchor,
    String? iconImage,
    List<double?>? iconOffset,
    double? iconRotate,
    double? iconSize,
    double? symbolSortKey,
    TextAnchor? textAnchor,
    String? textField,
    TextJustify? textJustify,
    double? textLetterSpacing,
    double? textMaxWidth,
    List<double?>? textOffset,
    double? textRadialOffset,
    double? textRotate,
    double? textSize,
    TextTransform? textTransform,
    int? iconColor,
    double? iconHaloBlur,
    int? iconHaloColor,
    double? iconHaloWidth,
    double? iconOpacity,
    int? textColor,
    double? textHaloBlur,
    int? textHaloColor,
    double? textHaloWidth,
    double? textOpacity,
  }) =>
      PointAnnotation(
        id: id,
        geometry: geometry ?? this.geometry,
        image: image ?? this.image,
        iconAnchor: iconAnchor ?? this.iconAnchor,
        iconImage: iconImage ?? this.iconImage,
        iconOffset: iconOffset ?? this.iconOffset,
        iconRotate: iconRotate ?? this.iconRotate,
        iconSize: iconSize ?? this.iconSize,
        symbolSortKey: symbolSortKey ?? this.symbolSortKey,
        textAnchor: textAnchor ?? this.textAnchor,
        textField: textField ?? this.textField,
        textJustify: textJustify ?? this.textJustify,
        textLetterSpacing: textLetterSpacing ?? this.textLetterSpacing,
        textMaxWidth: textMaxWidth ?? this.textMaxWidth,
        textOffset: textOffset ?? this.textOffset,
        textRadialOffset: textRadialOffset ?? this.textRadialOffset,
        textRotate: textRotate ?? this.textRotate,
        textSize: textSize ?? this.textSize,
        textTransform: textTransform ?? this.textTransform,
        iconColor: iconColor ?? this.iconColor,
        iconHaloBlur: iconHaloBlur ?? this.iconHaloBlur,
        iconHaloColor: iconHaloColor ?? this.iconHaloColor,
        iconHaloWidth: iconHaloWidth ?? this.iconHaloWidth,
        iconOpacity: iconOpacity ?? this.iconOpacity,
        textColor: textColor ?? this.textColor,
        textHaloBlur: textHaloBlur ?? this.textHaloBlur,
        textHaloColor: textHaloColor ?? this.textHaloColor,
        textHaloWidth: textHaloWidth ?? this.textHaloWidth,
        textOpacity: textOpacity ?? this.textOpacity,
      );

It worked well on android before. Alternatively, I also used this code, but the icon still disappears on update:

 busAnnotation?.iconImage = randomString;
 //...
 busAnnotation
        ?..geometry = position.toPoint().toJson()
        ..image = _tmpIcon;
thetranquila commented 3 days ago

I get this bug here, not on iconImage:

var response = http.get(Uri.parse(currentDynamic?.icon?.src ?? ""));
dynamicAnnotation?.image = response.bodyBytes.buffer.asUint8List();
pointAnnotationManager?.update(dynamicAnnotation);