MAPSJS-2660: BoundsGenerator cleanup.

- Move sphere view bounds and planar view bounds generation to separate classes. - Change interface to avoid having to change projection and wrapping both in MapView and BoundsGenerator. Signed-off-by: Andres Mandado <andres.mandado-almajano@here.com>
heremaps · Aug 27, 2021 · a417725 · a417725
1 parent cdb3d50
commit a417725
Show file tree

Hide file tree

Showing 6 changed files with 680 additions and 600 deletions.
diff --git a/@here/harp-examples/src/bounds-generation.ts b/@here/harp-examples/src/bounds-generation.ts
@@ -132,11 +132,7 @@ export namespace BoundsExample {
 
         addVectorTileDataSource(map);
         const featuresDataSource = addFeaturesDataSource(map, []);
-        const boundsGenerator = new BoundsGenerator(
-            map.camera,
-            map.projection,
-            map.tileWrappingEnabled
-        );
+        const boundsGenerator = new BoundsGenerator(map);
 
         let bounds: GeoPolygon | undefined;
         let showBoundingBox: boolean = false;
@@ -159,7 +155,7 @@ export namespace BoundsExample {
                     map.lookAt({ bounds });
                     break;
                 case "p":
-                    boundsGenerator.projection = map.projection =
+                    map.projection =
                         map.projection === mercatorProjection
                             ? sphereProjection
                             : mercatorProjection;
@@ -192,7 +188,6 @@ export namespace BoundsExample {
                     break;
                 case "w":
                     mapView.tileWrappingEnabled = !mapView.tileWrappingEnabled;
-                    boundsGenerator.tileWrappingEnabled = mapView.tileWrappingEnabled;
                     mapView.update();
                     break;
                 default: