From d7cef1d7907d93475c2db637c8713961e563ae94 Mon Sep 17 00:00:00 2001
From: tunglinn <tunglin@yahoo.com>
Date: Wed, 10 Jan 2024 11:12:23 -0800
Subject: [PATCH] Improve snapToCity

---
 src/js/setUpSite.js | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/js/setUpSite.js b/src/js/setUpSite.js
index e946761..7c736b3 100644
--- a/src/js/setUpSite.js
+++ b/src/js/setUpSite.js
@@ -165,8 +165,7 @@ const generateScorecard = (scoreCardEntry) => {
  * @param cityProperties: An object with a `layout` key (Leaflet value) and keys
  *    representing the score card properties stored in `score-cards.json`.
  */
-const snapToCity = async (map, cityProperties) => {
-  const { layer } = cityProperties;
+const snapToCity = async (map, layer) => {
   map.fitBounds(layer.getBounds());
 };
 
@@ -239,7 +238,7 @@ const setUpCitiesLayer = async (map) => {
   const cityToggleElement = document.getElementById("city-choice");
   cityToggleElement.addEventListener("change", () => {
     const cityId = cityToggleElement.value;
-    snapToCity(map, cities[cityId]);
+    snapToCity(map, cities[cityId].layer);
   });
 
   // Set up map to update when user clicks within a city's boundary
@@ -248,14 +247,14 @@ const setUpCitiesLayer = async (map) => {
     if (currentZoom > 7) {
       const cityId = e.sourceTarget.feature.properties.id;
       cityToggleElement.value = cityId;
-      snapToCity(map, cities[cityId]);
+      snapToCity(map, cities[cityId].layer);
     }
   });
 
   // Load initial city.
   const cityId = cityToggleElement.value;
   setUpAutoScorecard(map, cities);
-  snapToCity(map, cities[cityId]);
+  snapToCity(map, cities[cityId].layer);
   setScorecard(cityId, cities[cityId]);
 };