From 87403b561d6e5401b468438f6883f955285cd1d0 Mon Sep 17 00:00:00 2001
From: Hanbyul Jo <hanbyul@nyu.edu>
Date: Thu, 21 Sep 2023 11:20:24 -0400
Subject: [PATCH] Clamp bbox

---
 .../components/common/mapbox/layers/utils.ts  | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/app/scripts/components/common/mapbox/layers/utils.ts b/app/scripts/components/common/mapbox/layers/utils.ts
index 2e33ba3b4..97fecce74 100644
--- a/app/scripts/components/common/mapbox/layers/utils.ts
+++ b/app/scripts/components/common/mapbox/layers/utils.ts
@@ -1,7 +1,7 @@
 import { FunctionComponent, useEffect } from 'react';
 import { Feature } from 'geojson';
 import { Map as MapboxMap } from 'mapbox-gl';
-import { defaultsDeep } from 'lodash';
+import { defaultsDeep, clamp } from 'lodash';
 import axios, { Method } from 'axios';
 import {
   eachDayOfInterval,
@@ -412,13 +412,21 @@ type OptionalBbox = number[] | undefined | null;
 
 /**
  * MapboxGL requires maxX value to be 180, minX -180, maxY 90, minY -90
+ * While some of the datasets having bbox going above it.
  * @param bounds Bounding box to fit layer
  * @returns Boolean
  */
 
-function isBboxToFitValid(bounds) {
+function clampBbox(
+  bounds: [number, number, number, number]
+): [number, number, number, number] {
   const [minX, minY, maxX, maxY] = bounds;
-  return minX >= -180 && maxX <= 180 && minY >= -90 && maxY <= 90;
+  return [
+    clamp(minX, -180, 180),
+    clamp(minY, -90, 90),
+    clamp(maxX, -180, 180),
+    clamp(maxY, -90, 90)
+  ];
 }
 
 /**
@@ -445,8 +453,9 @@ export function useFitBbox(
       | [number, number, number, number]
       | undefined;
 
-    if (bounds?.length && isBboxToFitValid(bounds)) {
-      mapInstance.fitBounds(bounds, { padding: FIT_BOUNDS_PADDING });
+    if (bounds?.length) {
+      const clampedBbox = clampBbox(bounds);
+      mapInstance.fitBounds(clampedBbox, { padding: FIT_BOUNDS_PADDING });
     }
   }, [mapInstance, isUserPositionSet, initialBbox, stacBbox]);
 }