@nativescript-community/ui-mapbox 7.0.7 → 7.0.9

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [7.0.9](https://github.com/nativescript-community/ui-mapbox/compare/v7.0.8...v7.0.9) (2025-11-04)
+
+### Bug Fixes
+
+* **ios:** removed many unwanted error message (no actual error) ([f67277c](https://github.com/nativescript-community/ui-mapbox/commit/f67277c1ee0092eab4a2067b3d765c2aa2410246))
+
+## [7.0.8](https://github.com/nativescript-community/ui-mapbox/compare/v7.0.7...v7.0.8) (2025-11-03)
+
+**Note:** Version bump only for package @nativescript-community/ui-mapbox
+
 ## [7.0.7](https://github.com/nativescript-community/ui-mapbox/compare/v7.0.6...v7.0.7) (2025-10-30)
 
 **Note:** Version bump only for package @nativescript-community/ui-mapbox

package/index.android.d.ts

@@ -372,7 +372,7 @@ export declare class Mapbox extends MapboxCommon implements MapboxApi {
      */
     trackUser(options: TrackUserOptions, nativeMap?: any): Promise<void>;
     private static getAndroidColor;
-    _getMapStyle(input: any): any;
+    _getMapStyle(input?: string): any;
     /**
      * Mapbox Map Options
      *

package/index.android.js

@@ -2588,7 +2588,7 @@ export class Mapbox extends MapboxCommon {
             return android.graphics.Color.BLACK;
         }
     }
-    _getMapStyle(input) {
+    _getMapStyle(input = 'streets') {
         if (Trace.isEnabled()) {
             CLog(CLogTypes.info, '_getMapStyle(): top with input:', input);
         }

package/index.ios.js

@@ -145,8 +145,8 @@ export class MapboxView extends MapboxViewBase {
     }
     async disposeNativeView() {
         this.nativeView.owner = null;
-        if (this.mapbox)
-            await this.mapbox.destroy();
+        this.mapbox?.destroy();
+        this.mapbox = null;
         super.disposeNativeView();
     }
     getMapboxApi() {
@@ -398,7 +398,7 @@ export class Mapbox extends MapboxCommon {
                 }));
                 nativeMap.autoresizingMask = 2 /* UIViewAutoresizing.FlexibleWidth */ | 16 /* UIViewAutoresizing.FlexibleHeight */;
                 _markers = [];
-                this.addMarkers(settings.markers);
+                this.addMarkers(settings.markers || []);
                 // setTimeout(() => view.addSubview(nativeMap), 0);
                 view.addSubview(nativeMap);
                 this.setMapboxViewInstance(nativeMap);
@@ -463,24 +463,24 @@ export class Mapbox extends MapboxCommon {
                     this._programmaticMapView = null;
                 }
                 const bridge = MapboxBridge.bridgeFor(theMap);
-                bridge.destroy();
-                if (bridge === this.bridgeInstance) {
-                    this.bridgeInstance = null;
-                }
-                try {
-                    this._observerTokens.forEach((t) => {
-                        try {
-                            NSNotificationCenter.defaultCenter.removeObserver(t);
-                        }
-                        catch (e) {
-                            console.error(e, e.stack);
-                        }
-                    });
-                    this._observerTokens = [];
+                if (Trace.isEnabled()) {
+                    CLog(CLogTypes.info, 'destroy():', theMap, bridge);
                 }
-                catch (e) {
-                    console.error(e, e.stack);
+                if (bridge) {
+                    bridge.destroy();
+                    if (bridge === this.bridgeInstance) {
+                        this.bridgeInstance = null;
+                    }
                 }
+                this._observerTokens.forEach((t) => {
+                    try {
+                        NSNotificationCenter.defaultCenter.removeObserver(t);
+                    }
+                    catch (e) {
+                        console.error(e, e.stack);
+                    }
+                });
+                this._observerTokens = [];
                 if (this._reusableCalloutView) {
                     this._reusableCalloutView._tearDownUI();
                     this._reusableCalloutView = null;
@@ -1034,8 +1034,14 @@ export class Mapbox extends MapboxCommon {
                     reject('No bridge available');
                     return;
                 }
-                const styleStr = typeof style === 'string' ? style : style.toString();
+                if (Trace.isEnabled()) {
+                    CLog(CLogTypes.info, 'setMapStyle():', style);
+                }
+                const styleStr = (typeof style === 'string' ? style : style.toString()) ?? 'streets';
                 b.setStyle(styleStr, (success, error) => {
+                    if (Trace.isEnabled()) {
+                        CLog(CLogTypes.info, 'setMapStyle():', style, 'done:', success, error);
+                    }
                     if (success)
                         resolve();
                     else
@@ -1043,6 +1049,7 @@ export class Mapbox extends MapboxCommon {
                 });
             }
             catch (ex) {
+                console.error('setMapStyle():', ex, ex['stack']);
                 reject(ex);
             }
         });
@@ -1399,6 +1406,9 @@ export class Mapbox extends MapboxCommon {
         const marker = this._markers.find((m) => `${m.id}` === markerId);
         if (!marker)
             return;
+        if (!!marker.onTap?.(marker)) {
+            return;
+        }
         if (marker === this.selectedMarker) {
             this.deselectMarker(marker);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@nativescript-community/ui-mapbox",
-    "version": "7.0.7",
+    "version": "7.0.9",
     "description": "Interactive, thoroughly customizable maps powered by vector tiles and OpenGL.",
     "main": "index",
     "typings": "index.d.ts",
@@ -55,5 +55,5 @@
     "dependencies": {
         "@nativescript-community/perms": "^3.0.4"
     },
-    "gitHead": "d615644d576bcd0c702d1a7704157360a9d1d02c"
+    "gitHead": "997b1bd2abe92b1e9e4871ff58e2dc5f50513b79"
 }

package/platforms/ios/src/MapboxBridge.swift

@@ -592,7 +592,7 @@ public class MapboxBridge: NSObject {
         guard let mv = mapView else { completion(false, NSError(domain: "MapboxBridge", code: 1, userInfo: [NSLocalizedDescriptionKey: "No map available"])); return }
         mv.mapboxMap.onStyleLoaded.observeNext { _ in completion(true, nil) }.store(in: &cancelables)
         mv.mapboxMap.loadStyle(getMapStyleURI(styleURIorURL));
-        completion(false, NSError(domain: "MapboxBridge", code: 2, userInfo: [NSLocalizedDescriptionKey: "Invalid style string"]))
+        // completion(false, NSError(domain: "MapboxBridge", code: 2, userInfo: [NSLocalizedDescriptionKey: "Invalid style string"]))
     }
     
     // MARK: - Camera / viewport / animateCamera
@@ -953,14 +953,14 @@ public class MapboxBridge: NSObject {
         return true
         
     }
-    @objc public func removePolygons(_ _ids: [String]?) {
-        guard let manager = polygonAnnotationManager else { return }
+    @objc public func removePolygons(_ _ids: [String]?) -> Bool {
+        guard let manager = polygonAnnotationManager else { return false }
         guard let ids = _ids else {
             manager.annotations.removeAll()
             if let outlineManager = polygonOutlineAnnotationManager {
                 outlineManager.annotations.removeAll()
             }
-            return
+            return true
         }
         //        guard let data = idsJSON!.data(using: .utf8) else { return }
         //        guard let ids = try? JSONSerialization.jsonObject(with: data, options: []) as! [String] else { return }
@@ -977,18 +977,20 @@ public class MapboxBridge: NSObject {
         if let outlineManager = polygonOutlineAnnotationManager {
             outlineManager.annotations.removeAll { idSet.contains($0.id) }
         }
+        return true
     }
     
-    @objc public func removePolylines(_ _ids: [String]?) {
-        guard let manager = polylineAnnotationManager else { return }
+    @objc public func removePolylines(_ _ids: [String]?) -> Bool {
+        guard let manager = polylineAnnotationManager else { return false }
         guard let ids = _ids else {
             manager.annotations.removeAll()
-            return
+            return true
         }
         
         let idSet = Set<String>(ids)
         
         manager.annotations.removeAll { idSet.contains($0.id) }
+        return true
     }
     
     enum GeoJSONSourceUpdateError: Error {