react-native-maps 2.0.0-beta.11 → 2.0.0-beta.13

package/android/src/main/java/com/rnmaps/maps/MapManager.java

@@ -311,7 +311,7 @@ public class MapManager extends ViewGroupManager<MapView> {
         "onIndoorLevelActivated", MapBuilder.of("registrationName", "onIndoorLevelActivated"),
         "onIndoorBuildingFocused", MapBuilder.of("registrationName", "onIndoorBuildingFocused"),
         "onDoublePress", MapBuilder.of("registrationName", "onDoublePress"),
-        "onMapLoaded", MapBuilder.of("registrationName", "onMapLoaded")
+        "onTilesRendered", MapBuilder.of("registrationName", "onTilesRendered")
     ));
 
     return map;

package/android/src/main/java/com/rnmaps/maps/MapView.java

@@ -404,7 +404,7 @@ public class MapView extends com.google.android.gms.maps.MapView implements Goog
     map.setOnMapLoadedCallback(new GoogleMap.OnMapLoadedCallback() {
       @Override public void onMapLoaded() {
         isMapLoaded = true;
-        manager.pushEvent(context, view, "onMapLoaded", new WritableNativeMap());
+        manager.pushEvent(context, view, "onTilesRendered", new WritableNativeMap());
         MapView.this.cacheView();
       }
     });

package/android/src/main/java/com/rnmaps/maps/MapViewModule.java

@@ -434,26 +434,34 @@ public class MapViewModule extends ReactContextBaseJavaModule {
           promise.reject("RNMMapView.map is not valid");
           return;
         }
-        LatLngBounds.Builder builder = new LatLngBounds.Builder();
-
         boolean addedPosition = false;
 
+        double maxLatitude = -Double.MAX_VALUE;
+        double minLatitude = Double.MAX_VALUE;
+        double maxLongitude = -Double.MAX_VALUE;
+        double minLongitude = Double.MAX_VALUE;
+
         for (MapFeature feature : view.features) {
           if (feature instanceof MapMarker) {
-            Marker marker = (Marker) feature.getFeature();
-            builder.include(marker.getPosition());
+            LatLng position = ((Marker) feature.getFeature()).getPosition();
+            maxLatitude = Math.max(maxLatitude, position.latitude);
+            minLatitude = Math.min(minLatitude, position.latitude);
+            maxLongitude = Math.max(maxLongitude, position.longitude);
+            minLongitude = Math.min(minLongitude, position.longitude);
             addedPosition = true;
           }
           // TODO(lmr): may want to include shapes / etc.
         }
-        if (addedPosition) {
-          LatLngBounds bounds = builder.build();
-          CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, view.baseMapPadding);
-
-          if (edgePadding != null) {
-            view.map.setPadding(edgePadding.getInt("left"), edgePadding.getInt("top"),
-                    edgePadding.getInt("right"), edgePadding.getInt("bottom"));
-          }
+        if(!addedPosition) {
+          // nothing to do
+          promise.resolve(null);
+        } else {
+          Map<String, Double> paddedBoundary = padBoundary(view, maxLatitude, minLatitude, maxLongitude, minLongitude, edgePadding);
+          LatLngBounds bounds = new LatLngBounds(
+                  new LatLng(paddedBoundary.get("minLatitude"), paddedBoundary.get("minLongitude")), // southwest
+                  new LatLng(paddedBoundary.get("maxLatitude"), paddedBoundary.get("maxLongitude"))  // northeast
+          );
+          CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, 0);
 
           if (duration > 0) {
             view.map.animateCamera(cu, duration, new GoogleMap.CancelableCallback() {
@@ -491,7 +499,6 @@ public class MapViewModule extends ReactContextBaseJavaModule {
           promise.reject("RNMMapView.map is not valid");
           return;
         }
-        LatLngBounds.Builder builder = new LatLngBounds.Builder();
 
         String[] markerIDs = new String[markersIds.size()];
         for (int i = 0; i < markersIds.size(); i++) {
@@ -502,25 +509,35 @@ public class MapViewModule extends ReactContextBaseJavaModule {
 
         List<String> markerIDList = Arrays.asList(markerIDs);
 
+        double maxLatitude = -Double.MAX_VALUE;
+        double minLatitude = Double.MAX_VALUE;
+        double maxLongitude = -Double.MAX_VALUE;
+        double minLongitude = Double.MAX_VALUE;
+
         for (MapFeature feature : view.features) {
           if (feature instanceof MapMarker) {
             String identifier = ((MapMarker) feature).getIdentifier();
-            Marker marker = (Marker) feature.getFeature();
             if (markerIDList.contains(identifier)) {
-              builder.include(marker.getPosition());
+              LatLng position = ((Marker) feature.getFeature()).getPosition();
+              maxLatitude = Math.max(maxLatitude, position.latitude);
+              minLatitude = Math.min(minLatitude, position.latitude);
+              maxLongitude = Math.max(maxLongitude, position.longitude);
+              minLongitude = Math.min(minLongitude, position.longitude);
               addedPosition = true;
             }
           }
         }
 
-        if (addedPosition) {
-          LatLngBounds bounds = builder.build();
-          CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, view.baseMapPadding);
-
-          if (edgePadding != null) {
-            view.map.setPadding(edgePadding.getInt("left"), edgePadding.getInt("top"),
-                    edgePadding.getInt("right"), edgePadding.getInt("bottom"));
-          }
+        if(!addedPosition) {
+          // nothing to do
+          promise.resolve(null);
+        } else {
+          Map<String, Double> paddedBoundary = padBoundary(view, maxLatitude, minLatitude, maxLongitude, minLongitude, edgePadding);
+          LatLngBounds bounds = new LatLngBounds(
+                  new LatLng(paddedBoundary.get("minLatitude"), paddedBoundary.get("minLongitude")), // southwest
+                  new LatLng(paddedBoundary.get("maxLatitude"), paddedBoundary.get("maxLongitude"))  // northeast
+          );
+          CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, 0);
 
           if (duration > 0) {
             view.map.animateCamera(cu, duration, new GoogleMap.CancelableCallback() {
@@ -558,21 +575,28 @@ public class MapViewModule extends ReactContextBaseJavaModule {
           return;
         }
 
-        LatLngBounds.Builder builder = new LatLngBounds.Builder();
+        double maxLatitude = -Double.MAX_VALUE;
+        double minLatitude = Double.MAX_VALUE;
+        double maxLongitude = -Double.MAX_VALUE;
+        double minLongitude = Double.MAX_VALUE;
 
         for (int i = 0; i < coordinates.size(); i++) {
           ReadableMap latLng = coordinates.getMap(i);
           double lat = latLng.getDouble("latitude");
           double lng = latLng.getDouble("longitude");
-          builder.include(new LatLng(lat, lng));
+          maxLatitude = Math.max(maxLatitude, lat);
+          minLatitude = Math.min(minLatitude, lat);
+          maxLongitude = Math.max(maxLongitude, lng);
+          minLongitude = Math.min(minLongitude, lng);
         }
 
-        LatLngBounds bounds = builder.build();
-        CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, view.baseMapPadding);
+        Map<String, Double> paddedBoundary = padBoundary(view, maxLatitude, minLatitude, maxLongitude, minLongitude, edgePadding);
+        LatLngBounds bounds = new LatLngBounds(
+                new LatLng(paddedBoundary.get("minLatitude"), paddedBoundary.get("minLongitude")), // southwest
+                new LatLng(paddedBoundary.get("maxLatitude"), paddedBoundary.get("maxLongitude"))  // northeast
+        );
 
-        if (edgePadding != null) {
-          view.appendMapPadding(edgePadding.getInt("left"), edgePadding.getInt("top"), edgePadding.getInt("right"), edgePadding.getInt("bottom"));
-        }
+        CameraUpdate cu = CameraUpdateFactory.newLatLngBounds(bounds, 0);
 
         if (duration > 0) {
           view.map.animateCamera(cu, duration, new GoogleMap.CancelableCallback() {
@@ -590,9 +614,29 @@ public class MapViewModule extends ReactContextBaseJavaModule {
           view.map.moveCamera(cu);
           promise.resolve(null);
         }
-        // Move the google logo to the default base padding value.
-        view.map.setPadding(view.baseLeftMapPadding, view.baseTopMapPadding, view.baseRightMapPadding, view.baseBottomMapPadding);
       }
     });
   }
+
+  private Map<String, Double> padBoundary(MapView mapView, double maxLat, double minLat, double maxLng, double minLng, ReadableMap edgePadding) {
+    double latitudeDelta = maxLat - minLat;
+    double longitudeDelta = maxLng - minLng;
+    float displayDensity = context.getResources().getDisplayMetrics().density;
+    double mapViewHeight = mapView.getHeight() / displayDensity;
+    double mapViewWidth = mapView.getWidth() / displayDensity;
+    double latPerHeight = latitudeDelta / mapViewHeight;
+    double lngPerWidth = longitudeDelta / mapViewWidth;
+    double paddedMaxLatitude = maxLat + latPerHeight * edgePadding.getInt("top");
+    double paddedMinLatitude = minLat - latPerHeight * edgePadding.getInt("bottom");
+    double paddedMaxLongitude = maxLng + lngPerWidth * edgePadding.getInt("right");
+    double paddedMinLongitude = minLng - lngPerWidth * edgePadding.getInt("left");
+
+    Map<String,Double> paddedBoundary = new HashMap<String,Double>();
+    paddedBoundary.put("maxLatitude", paddedMaxLatitude);
+    paddedBoundary.put("minLatitude", paddedMinLatitude);
+    paddedBoundary.put("maxLongitude", paddedMaxLongitude);
+    paddedBoundary.put("minLongitude", paddedMinLongitude);
+
+    return paddedBoundary;
+  }
 }

package/ios/GoogleMaps/RNMGoogleMap.h

@@ -18,62 +18,61 @@
 @interface RNMGoogleMap : GMSMapView
 
 // TODO: don't use MK region?
-@property (nonatomic, weak) RCTBridge *bridge;
+@property (nonatomic, assign) BOOL pitchEnabled;
+@property (nonatomic, assign) BOOL rotateEnabled;
+@property (nonatomic, assign) BOOL scrollDuringRotateOrZoomEnabled;
+@property (nonatomic, assign) BOOL scrollEnabled;
+@property (nonatomic, assign) BOOL showsBuildings;
+@property (nonatomic, assign) BOOL showsCompass;
+@property (nonatomic, assign) BOOL showsIndoorLevelPicker;
+@property (nonatomic, assign) BOOL showsIndoors;
+@property (nonatomic, assign) BOOL showsMyLocationButton;
+@property (nonatomic, assign) BOOL showsTraffic;
+@property (nonatomic, assign) BOOL showsUserLocation;
+@property (nonatomic, assign) BOOL zoomEnabled;
+@property (nonatomic, assign) BOOL zoomTapEnabled;
+@property (nonatomic, assign) GMSCameraPosition *cameraProp;   // Because the base class already has a "camera" prop.
 @property (nonatomic, assign) MKCoordinateRegion initialRegion;
 @property (nonatomic, assign) MKCoordinateRegion region;
-@property (nonatomic, assign) GMSCameraPosition *cameraProp;   // Because the base class already has a "camera" prop.
-@property (nonatomic, strong) GMSCameraPosition *initialCamera;
 @property (nonatomic, assign) NSString *customMapStyleString;
-@property (nonatomic, assign) UIEdgeInsets mapPadding;
+@property (nonatomic, assign) NSString *kmlSrc;
 @property (nonatomic, assign) NSString *paddingAdjustmentBehaviorString;
-@property (nonatomic, copy) RCTBubblingEventBlock onMapReady;
-@property (nonatomic, copy) RCTBubblingEventBlock onMapLoaded;
+@property (nonatomic, assign) UIEdgeInsets mapPadding;
+@property (nonatomic, copy) RCTBubblingEventBlock onChange;
 @property (nonatomic, copy) RCTBubblingEventBlock onKmlReady;
-@property (nonatomic, copy) RCTBubblingEventBlock onPress;
 @property (nonatomic, copy) RCTBubblingEventBlock onLongPress;
-@property (nonatomic, copy) RCTBubblingEventBlock onPanDrag;
-@property (nonatomic, copy) RCTBubblingEventBlock onUserLocationChange;
+@property (nonatomic, copy) RCTBubblingEventBlock onMapReady;
 @property (nonatomic, copy) RCTBubblingEventBlock onMarkerPress;
-@property (nonatomic, copy) RCTBubblingEventBlock onChange;
+@property (nonatomic, copy) RCTBubblingEventBlock onPanDrag;
 @property (nonatomic, copy) RCTBubblingEventBlock onPoiClick;
+@property (nonatomic, copy) RCTBubblingEventBlock onPress;
+@property (nonatomic, copy) RCTBubblingEventBlock onTilesRendered;
+@property (nonatomic, copy) RCTBubblingEventBlock onUserLocationChange;
+@property (nonatomic, copy) RCTDirectEventBlock onIndoorBuildingFocused;
+@property (nonatomic, copy) RCTDirectEventBlock onIndoorLevelActivated;
 @property (nonatomic, copy) RCTDirectEventBlock onRegionChange;
 @property (nonatomic, copy) RCTDirectEventBlock onRegionChangeComplete;
-@property (nonatomic, copy) RCTDirectEventBlock onIndoorLevelActivated;
-@property (nonatomic, copy) RCTDirectEventBlock onIndoorBuildingFocused;
+@property (nonatomic, strong) GMSCameraPosition *initialCamera;
+@property (nonatomic, strong) NSMutableArray *circles;
+@property (nonatomic, strong) NSMutableArray *heatmaps;
 @property (nonatomic, strong) NSMutableArray *markers;
+@property (nonatomic, strong) NSMutableArray *overlays;
 @property (nonatomic, strong) NSMutableArray *polygons;
 @property (nonatomic, strong) NSMutableArray *polylines;
-@property (nonatomic, strong) NSMutableArray *circles;
-@property (nonatomic, strong) NSMutableArray *heatmaps;
 @property (nonatomic, strong) NSMutableArray *tiles;
-@property (nonatomic, strong) NSMutableArray *overlays;
-
-@property (nonatomic, assign) BOOL showsBuildings;
-@property (nonatomic, assign) BOOL showsTraffic;
-@property (nonatomic, assign) BOOL showsCompass;
-@property (nonatomic, assign) BOOL scrollEnabled;
-@property (nonatomic, assign) BOOL zoomEnabled;
-@property (nonatomic, assign) BOOL rotateEnabled;
-@property (nonatomic, assign) BOOL scrollDuringRotateOrZoomEnabled;
-@property (nonatomic, assign) BOOL pitchEnabled;
-@property (nonatomic, assign) BOOL zoomTapEnabled;
-@property (nonatomic, assign) BOOL showsUserLocation;
-@property (nonatomic, assign) BOOL showsMyLocationButton;
-@property (nonatomic, assign) BOOL showsIndoors;
-@property (nonatomic, assign) BOOL showsIndoorLevelPicker;
-@property (nonatomic, assign) NSString *kmlSrc;
+@property (nonatomic, weak) RCTBridge *bridge;
 
-- (void)didPrepareMap;
-- (void)mapViewDidFinishTileRendering;
 - (BOOL)didTapMarker:(GMSMarker *)marker;
-- (void)didTapPolyline:(GMSPolyline *)polyline;
-- (void)didTapPolygon:(GMSPolygon *)polygon;
-- (void)didTapAtCoordinate:(CLLocationCoordinate2D)coordinate;
-- (void)didLongPressAtCoordinate:(CLLocationCoordinate2D)coordinate;
+- (NSArray *)getMapBoundaries;
 - (void)didChangeCameraPosition:(GMSCameraPosition *)position isGesture:(BOOL)isGesture;
-- (void)idleAtCameraPosition:(GMSCameraPosition *)position isGesture:(BOOL)isGesture;
+- (void)didLongPressAtCoordinate:(CLLocationCoordinate2D)coordinate;
+- (void)didPrepareMap;
+- (void)didTapAtCoordinate:(CLLocationCoordinate2D)coordinate;
 - (void)didTapPOIWithPlaceID:(NSString *)placeID name:(NSString *) name location:(CLLocationCoordinate2D) location;
-- (NSArray *)getMapBoundaries;
+- (void)didTapPolygon:(GMSPolygon *)polygon;
+- (void)didTapPolyline:(GMSPolyline *)polyline;
+- (void)idleAtCameraPosition:(GMSCameraPosition *)position isGesture:(BOOL)isGesture;
+- (void)mapViewDidFinishTileRendering;
 
 + (MKCoordinateRegion)makeGMSCameraPositionFromMap:(GMSMapView *)map andGMSCameraPosition:(GMSCameraPosition *)position;
 + (GMSCameraPosition*)makeGMSCameraPositionFromMap:(GMSMapView *)map andMKCoordinateRegion:(MKCoordinateRegion)region;

package/ios/GoogleMaps/RNMGoogleMap.m

@@ -330,7 +330,7 @@ id regionAsJSON(MKCoordinateRegion region) {
 }
 
 - (void)mapViewDidFinishTileRendering {
-  if (self.onMapLoaded) self.onMapLoaded(@{});
+  if (self.onTilesRendered) self.onTilesRendered(@{});
 }
 
 - (BOOL)didTapMarker:(GMSMarker *)marker {

package/ios/GoogleMaps/RNMGoogleMapManager.m

@@ -81,7 +81,6 @@ RCT_EXPORT_VIEW_PROPERTY(onIndoorBuildingFocused, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onIndoorLevelActivated, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onKmlReady, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onLongPress, RCTBubblingEventBlock)
-RCT_EXPORT_VIEW_PROPERTY(onMapLoaded, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onMapReady, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onMarkerPress, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onPanDrag, RCTBubblingEventBlock)
@@ -89,6 +88,7 @@ RCT_EXPORT_VIEW_PROPERTY(onPoiClick, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onPress, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onRegionChange, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onRegionChangeComplete, RCTDirectEventBlock)
+RCT_EXPORT_VIEW_PROPERTY(onTilesRendered, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onUserLocationChange, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(pitchEnabled, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(region, MKCoordinateRegion)

package/ios/GoogleMaps/RNMGoogleMapViewModule.m

@@ -261,17 +261,13 @@ RCT_EXPORT_METHOD(fitToElements:(nonnull NSNumber *)reactTag
         
         GMSCameraUpdate* cameraUpdate;
         
-        if ([edgePadding count] != 0) {
-            // Set Map viewport
-            CGFloat top = [RCTConvert CGFloat:edgePadding[@"top"]];
-            CGFloat right = [RCTConvert CGFloat:edgePadding[@"right"]];
-            CGFloat bottom = [RCTConvert CGFloat:edgePadding[@"bottom"]];
-            CGFloat left = [RCTConvert CGFloat:edgePadding[@"left"]];
+        // Set Map viewport
+        CGFloat top = [RCTConvert CGFloat:edgePadding[@"top"]];
+        CGFloat right = [RCTConvert CGFloat:edgePadding[@"right"]];
+        CGFloat bottom = [RCTConvert CGFloat:edgePadding[@"bottom"]];
+        CGFloat left = [RCTConvert CGFloat:edgePadding[@"left"]];
             
-            cameraUpdate = [GMSCameraUpdate fitBounds:bounds withEdgeInsets:UIEdgeInsetsMake(top, left, bottom, right)];
-        } else {
-            cameraUpdate = [GMSCameraUpdate fitBounds:bounds withPadding:55.0f];
-        }
+        cameraUpdate = [GMSCameraUpdate fitBounds:bounds withEdgeInsets:UIEdgeInsetsMake(top, left, bottom, right)];
       if (duration > 0.0f) {
         [CATransaction begin];
         [CATransaction setCompletionBlock:^{

package/ios/Maps/RNMMap.h

@@ -66,6 +66,7 @@ extern const NSInteger RNMMapMaxZoomLevel;
 @property (nonatomic, copy) RCTDirectEventBlock onMarkerDragEnd;
 @property (nonatomic, copy) RCTDirectEventBlock onCalloutPress;
 @property (nonatomic, copy) RCTDirectEventBlock onRegionChange;
+@property (nonatomic, copy) RCTBubblingEventBlock onTilesRendered;
 @property (nonatomic, copy) RCTBubblingEventBlock onUserLocationChange;
 
 - (void)cacheViewIfNeeded;

package/ios/Maps/RNMMapManager.m

@@ -111,6 +111,7 @@ RCT_EXPORT_VIEW_PROPERTY(onMarkerPress, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onMarkerSelect, RCTDirectEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onPanDrag, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onPress, RCTBubblingEventBlock)
+RCT_EXPORT_VIEW_PROPERTY(onTilesRendered, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(onUserLocationChange, RCTBubblingEventBlock)
 RCT_EXPORT_VIEW_PROPERTY(pitchEnabled, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(rotateEnabled, BOOL)
@@ -551,9 +552,12 @@ static int kDragCenterContext;
     [mapView beginLoading];
 }
 
-- (void)mapViewDidFinishRenderingMap:(RNMMap *)mapView fullyRendered:(BOOL)fullyRendered
+- (void)mapViewDidFinishRenderingMap:(RNMMap *)mapView fullyRendered:(__unused BOOL)fullyRendered
 {
     [mapView finishLoading];
+    if (mapView.onTilesRendered) {
+        mapView.onTilesRendered(@{});
+    }
 }
 
 #pragma mark Private

package/ios/Maps/RNMMapViewModule.m

@@ -287,21 +287,62 @@ RCT_EXPORT_METHOD(fitToElements:(nonnull NSNumber *)reactTag
             RCTLogError(@"Invalid view returned from registry, expecting RNMMap, got: %@", view);
         } else {
             RNMMap *mapView = (RNMMap *)view;
+            
+            if(mapView.annotations.count < 1) {
+                resolve(nil);
+            } else {
+                CLLocationDegrees minLatitude = DBL_MAX;
+                CLLocationDegrees maxLatitude = -DBL_MAX;
+                CLLocationDegrees minLongitude = DBL_MAX;
+                CLLocationDegrees maxLongitude = -DBL_MAX;
+                
+                for(id<MKAnnotation> annotation in mapView.annotations) {
+                    double annotationLat = annotation.coordinate.latitude;
+                    double annotationLong = annotation.coordinate.longitude;
+                    minLatitude = fmin(annotationLat, minLatitude);
+                    maxLatitude = fmax(annotationLat, maxLatitude);
+                    minLongitude = fmin(annotationLong, minLongitude);
+                    maxLongitude = fmax(annotationLong, maxLongitude);
+                }
+                
+                double latitudeDelta = maxLatitude - minLatitude;
+                double longitudeDelta = maxLongitude - minLongitude;
+                
+                double mapViewHeight = mapView.bounds.size.height;
+                double mapViewWidth = mapView.bounds.size.width;
+                
+                double latPerHeight = latitudeDelta / mapViewHeight;
+                double lngPerWidth = longitudeDelta / mapViewWidth;
+                
+                CGFloat topPadding = [RCTConvert CGFloat:edgePadding[@"top"]];
+                CGFloat rightPadding = [RCTConvert CGFloat:edgePadding[@"right"]];
+                CGFloat bottomPadding = [RCTConvert CGFloat:edgePadding[@"bottom"]];
+                CGFloat leftPadding = [RCTConvert CGFloat:edgePadding[@"left"]];
+                
+                maxLatitude = maxLatitude + topPadding * latPerHeight;
+                minLatitude = minLatitude - bottomPadding * latPerHeight;
+                maxLongitude = maxLongitude + rightPadding * lngPerWidth;
+                minLongitude = minLongitude - leftPadding * lngPerWidth;
+                
+                MKCoordinateRegion region;
+                region.center.latitude = (minLatitude + maxLatitude) / 2;
+                region.center.longitude = (minLongitude + maxLongitude) / 2;
+                // if fitting a single marker or if all markers are aligned, the delta will be zero (infinite zoom)
+                // this will cause mapkit not to zoom at all. Picking an arbitrary small number to circumvent
+                region.span.latitudeDelta = fmax(maxLatitude - minLatitude, 0.000001);
+                region.span.longitudeDelta = fmax(maxLongitude - minLongitude, 0.000001);
 
-            // TODO(lmr): we potentially want to include overlays here... and could concat the two arrays together.
-            dispatch_after(dispatch_time(DISPATCH_TIME_NOW, 0.1 * NSEC_PER_SEC), dispatch_get_main_queue(), ^{
                 if(duration > 0.0f) {
                     [RNMMap animateWithDuration:duration/1000 animations:^{
-                        [mapView showAnnotations:mapView.annotations animated:YES];
+                        [mapView setRegion:region animated:YES];
                     } completion:^(BOOL finished){
                         resolve(nil);
                     }];
                 } else {
-                    [mapView showAnnotations:mapView.annotations animated:NO];
+                    [mapView setRegion:region animated:NO];
                     resolve(nil);
                 }
-                
-            });
+            }
         }
     }];
 }
@@ -328,17 +369,62 @@ RCT_EXPORT_METHOD(fitToSuppliedMarkers:(nonnull NSNumber *)reactTag
             }];
 
             NSArray *filteredMarkers = [mapView.annotations filteredArrayUsingPredicate:filterMarkers];
+            
+            if(filteredMarkers.count < 1) {
+                resolve(nil);
+            } else {
+                CLLocationDegrees minLatitude = DBL_MAX;
+                CLLocationDegrees maxLatitude = -DBL_MAX;
+                CLLocationDegrees minLongitude = DBL_MAX;
+                CLLocationDegrees maxLongitude = -DBL_MAX;
+                
+                for(id<MKAnnotation> annotation in filteredMarkers) {
+                    double annotationLat = annotation.coordinate.latitude;
+                    double annotationLong = annotation.coordinate.longitude;
+                    minLatitude = fmin(annotationLat, minLatitude);
+                    maxLatitude = fmax(annotationLat, maxLatitude);
+                    minLongitude = fmin(annotationLong, minLongitude);
+                    maxLongitude = fmax(annotationLong, maxLongitude);
+                }
+                
+                double latitudeDelta = maxLatitude - minLatitude;
+                double longitudeDelta = maxLongitude - minLongitude;
+                
+                double mapViewHeight = mapView.bounds.size.height;
+                double mapViewWidth = mapView.bounds.size.width;
+                
+                double latPerHeight = latitudeDelta / mapViewHeight;
+                double lngPerWidth = longitudeDelta / mapViewWidth;
+                
+                CGFloat topPadding = [RCTConvert CGFloat:edgePadding[@"top"]];
+                CGFloat rightPadding = [RCTConvert CGFloat:edgePadding[@"right"]];
+                CGFloat bottomPadding = [RCTConvert CGFloat:edgePadding[@"bottom"]];
+                CGFloat leftPadding = [RCTConvert CGFloat:edgePadding[@"left"]];
+                
+                maxLatitude = maxLatitude + topPadding * latPerHeight;
+                minLatitude = minLatitude - bottomPadding * latPerHeight;
+                maxLongitude = maxLongitude + rightPadding * lngPerWidth;
+                minLongitude = minLongitude - leftPadding * lngPerWidth;
+                
+                MKCoordinateRegion region;
+                region.center.latitude = (minLatitude + maxLatitude) / 2;
+                region.center.longitude = (minLongitude + maxLongitude) / 2;
+                // if fitting a single marker or if all markers are aligned, the delta will be zero (infinite zoom)
+                // this will cause mapkit not to zoom at all. Picking an arbitrary small number to circumvent
+                region.span.latitudeDelta = fmax(maxLatitude - minLatitude, 0.000001);
+                region.span.longitudeDelta = fmax(maxLongitude - minLongitude, 0.000001);
 
-            if(duration > 0.0f) {
-                [RNMMap animateWithDuration:duration/1000 animations:^{
-                [mapView showAnnotations:filteredMarkers animated:YES];
-                } completion:^(BOOL finished){
-                    resolve(nil);
-                }];
+                if(duration > 0.0f) {
+                    [RNMMap animateWithDuration:duration/1000 animations:^{
+                        [mapView setRegion:region animated:YES];
+                    } completion:^(BOOL finished){
+                        resolve(nil);
+                    }];
                 } else {
-                [mapView showAnnotations:filteredMarkers animated:NO];
+                    [mapView setRegion:region animated:NO];
                     resolve(nil);
                 }
+            }
         }
     }];
 }

package/lib/MapView.d.ts

@@ -227,13 +227,6 @@ export type MapViewProps = ViewProps & {
      * @platform Android: Supported
      */
     onLongPress?: (event: LongPressEvent) => void;
-    /**
-     * Callback that is called when the map has finished rendering all tiles.
-     *
-     * @platform iOS: Google Maps only
-     * @platform Android: Supported
-     */
-    onMapLoaded?: (event: NativeSyntheticEvent<{}>) => void;
     /**
      * Callback that is called once the map is ready.
      *
@@ -329,6 +322,13 @@ export type MapViewProps = ViewProps & {
      * @platform Android: Supported
      */
     onRegionChangeComplete?: (region: Region, details: Details) => void;
+    /**
+     * Callback that is called when the map has finished rendering all tiles.
+     *
+     * @platform iOS: Supported
+     * @platform Android: Supported
+     */
+    onTilesRendered?: (event: NativeSyntheticEvent<{}>) => void;
     /**
      * Callback that is called when the underlying map figures our users current location
      * (coordinate also includes isFromMockProvider value for Android API 18 and above).

package/package.json

@@ -5,7 +5,7 @@
   "main": "lib/index.js",
   "author": "Leland Richardson <leland.m.richardson@gmail.com>",
   "homepage": "https://github.com/react-native-maps/react-native-maps#readme",
-  "version": "2.0.0-beta.11",
+  "version": "2.0.0-beta.13",
   "license": "MIT",
   "scripts": {
     "lint": "eslint . --max-warnings 0",