From 62669fc7de5ede5a325d12e449bfb259b2df594c Mon Sep 17 00:00:00 2001
From: Matt <matt@echo6.co>
Date: Sat, 2 May 2026 18:36:07 +0000
Subject: [PATCH] fix(map): Always zoom in to fit boundaries, never zoom out
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replace blanket z14 cap with zoom-in-only logic for boundary fitBounds:
- target.zoom > currentZoom → zoom IN to fit boundary (always allowed)
- target.zoom < currentZoom → skip fitBounds (never zoom out)
- target.zoom == currentZoom → allow pan to center

This means:
- Click Portland at z5 → fly in to ~z10 to show city boundary
- Click Idaho at z5 → fly in to ~z6 to show state boundary
- Click Portland at z15 → boundary draws, camera stays at z15

The z14 cap for selectedPlace flyTo (search results without boundary)
is preserved - that only affects the initial flyTo to coordinates.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/components/MapView.jsx | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/components/MapView.jsx b/src/components/MapView.jsx
index 2dbed45..bf61605 100644
--- a/src/components/MapView.jsx
+++ b/src/components/MapView.jsx
@@ -2421,16 +2421,24 @@ const MapView = forwardRef(function MapView(_, ref) {
               // Validate bounds before fitting
               if (minLng >= -180 && maxLng <= 180 && minLat >= -90 && maxLat <= 90 &&
                   minLng < maxLng && minLat < maxLat) {
-                // Only fit bounds if zoomed out (< z14). At z14+ just draw boundary silently.
+                const bounds = [[minLng, minLat], [maxLng, maxLat]]
                 const currentZoom = map.getZoom()
-                if (currentZoom < 14) {
-                  const bounds = [[minLng, minLat], [maxLng, maxLat]]
-                  map.fitBounds(bounds, {
-                    padding: 50,
-                    duration: 700,
-                    maxZoom: 16,
-                  })
+                const target = map.cameraForBounds(bounds, { padding: 50 })
+                
+                // Zoom-in only: allow zoom in to show boundary, never zoom out
+                // - target.zoom > currentZoom → zoom IN to fit (always allowed)
+                // - target.zoom < currentZoom → DON'T zoom out (skip fitBounds)
+                // - target.zoom == currentZoom → pan only (allowed)
+                if (!target || target.zoom < currentZoom) {
+                  // Would zoom out — just draw the boundary without moving camera
+                  return
                 }
+                
+                map.fitBounds(bounds, {
+                  padding: 50,
+                  duration: 700,
+                  maxZoom: 16,
+                })
               }
             }
           } catch (e) {