refactor: Only update text area when necessary

mbta · Jul 2, 2024 · 8022a49 · 8022a49
1 parent 88dbdeb
commit 8022a49
Showing 1 changed file with 19 additions and 16 deletions.
diff --git a/assets/src/components/detours/diversionPage.tsx b/assets/src/components/detours/diversionPage.tsx
@@ -106,23 +106,26 @@ export const DiversionPage = ({
     : {}
 
   useEffect(() => {
-    setTextArea(
-      [
-        `Detour ${routeName} ${routeDirection}`,
-        routeOrigin,
-        ,
-        "Connection Points:",
-        connectionPoints?.start?.name ?? "N/A",
-        connectionPoints?.end?.name ?? "N/A",
-        ,
-        `Missed Stops (${missedStops?.length}):`,
-        ...(missedStops?.map(({ name }) => name) ?? ["no stops"]),
-        ,
-        "Turn-by-Turn Directions:",
-        ...(extendedDirections?.map((v) => v.instruction) ?? []),
-      ].join("\n")
-    )
+    if (snapshot.matches({ "Detour Drawing": "Share Detour" })) {
+      setTextArea(
+        [
+          `Detour ${routeName} ${routeDirection}`,
+          routeOrigin,
+          ,
+          "Connection Points:",
+          connectionPoints?.start?.name ?? "N/A",
+          connectionPoints?.end?.name ?? "N/A",
+          ,
+          `Missed Stops (${missedStops?.length}):`,
+          ...(missedStops?.map(({ name }) => name) ?? ["no stops"]),
+          ,
+          "Turn-by-Turn Directions:",
+          ...(extendedDirections?.map((v) => v.instruction) ?? []),
+        ].join("\n")
+      )
+    }
   }, [
+    snapshot,
     routeName,
     routeDirection,
     routeOrigin,