Don't track bearing/pitch for view comparison

Author: camdecoster

src/plots/map/layout_defaults.js

@@ -48,20 +48,12 @@ function handleDefaults(containerIn, containerOut, coerce, opts) {
     // hold the exact values that were auto-computed last time. Skipped when
     // the user has explicitly opted out via `fitbounds: false`.
     // - `_fitBounds` contains the lon/lat coords for the geometry bounding box
-    // - `_fitView` contains the view attributes after the MapLibre auto-fit has been completed
-    const {
-        _fitView: { center: fitCenter, zoom: fitZoom, bearing: fitBearing, pitch: fitPitch } = {},
-        center,
-        zoom,
-        bearing,
-        pitch
-    } = containerIn;
-    const isFitView =
-        bearing === fitBearing &&
-        center?.lon === fitCenter?.lon &&
-        center?.lat === fitCenter?.lat &&
-        pitch === fitPitch &&
-        zoom === fitZoom;
+    //   and is used to store the values passed in to MapLibre for fitting
+    // - `_fitView` contains the view attributes after the MapLibre auto-fit
+    //   has been completed and is used to determine if the current view is
+    //   still the auto-fit view
+    const { _fitView: { center: fitCenter, zoom: fitZoom } = {}, center, zoom } = containerIn;
+    const isFitView = center?.lon === fitCenter?.lon && center?.lat === fitCenter?.lat && zoom === fitZoom;
     if (fitbounds && isFitView) {
         const fitBounds = getMapFitBounds(opts.fullData, opts.id);
         if (fitBounds) containerOut._fitBounds = fitBounds;

src/plots/map/map.js

@@ -241,15 +241,9 @@ function saveViewToLayout(self, opts) {
     opts._input.center = opts.center = center;
     opts._input.zoom = opts.zoom = zoom;
     // Use below to determine if the current view is still on the fit view during supplyDefaults.
-    // Bearing and pitch aren't set by auto-fit, but a user change to either signals
-    // they've engaged with the view and don't want it refit under them. `_fitView` represents
-    // the view attributes returned by MapLibre after the auto-fit has been completed.
-    opts._input._fitView = {
-        center: { ...center },
-        zoom,
-        bearing: opts._input.bearing,
-        pitch: opts._input.pitch
-    };
+    // `_fitView` represents the view attributes returned by MapLibre after the auto-fit has been
+    // completed.
+    opts._input._fitView = { center: { ...center }, zoom };
 }
 
 // Snapshot the current map view state for the Reset view modebar