Fix tree edit functionality and improve popup close button positioning

- Fix edit tree flow from map tooltips to main form
- Add checkForEditTree() to automatically load trees for editing from localStorage
- Simplify map.js editTree method to just store ID and redirect
- Fix Leaflet popup close button positioning with proper CSS
- Update service worker cache version to force asset refresh
- Update cache timestamps for map.js to ensure latest version delivery

Fixes the 'error fetching tree data' issue when editing trees from map view.
Users can now seamlessly edit trees directly from map marker popups.

static/js/tree-track-app.js

@@ -112,11 +112,30 @@ export class TreeTrackApp {
 
     async loadInitialData() {
         try {
+            // Check if we need to load a tree for editing
+            await this.checkForEditTree();
+            
+            // Load trees list
             await this.loadTrees();
         } catch (error) {
             console.error('Error loading initial data:', error);
         }
     }
+    
+    async checkForEditTree() {
+        const editTreeId = localStorage.getItem('editTreeId');
+        if (editTreeId) {
+            // Clear the localStorage item to prevent repeated loads
+            localStorage.removeItem('editTreeId');
+            
+            try {
+                await this.handleEditTree(parseInt(editTreeId));
+            } catch (error) {
+                console.error('Error loading tree for edit from localStorage:', error);
+                this.uiManager.showMessage('Error loading tree for editing: ' + error.message, 'error');
+            }
+        }
+    }
 
     async handleFormSubmit(e) {
         e.preventDefault();

static/map.html

@@ -451,6 +451,7 @@
             border-radius: var(--radius-lg);
             box-shadow: var(--shadow-xl);
             border: 1px solid var(--gray-200);
+            position: relative;
         }
 
         .leaflet-popup-tip {
@@ -461,6 +462,44 @@
         .leaflet-popup-content {
             margin: 0;
             line-height: 1.5;
+            overflow: hidden;
+        }
+        
+        /* Fix popup close button positioning */
+        .leaflet-popup-close-button {
+            position: absolute;
+            top: 8px;
+            right: 8px;
+            z-index: 10;
+            background: rgba(255, 255, 255, 0.9);
+            color: var(--gray-600);
+            border: 1px solid var(--gray-300);
+            border-radius: 50%;
+            width: 24px;
+            height: 24px;
+            display: flex;
+            align-items: center;
+            justify-content: center;
+            font-size: 14px;
+            line-height: 1;
+            cursor: pointer;
+            transition: all 0.15s ease;
+            text-decoration: none;
+            font-weight: 400;
+            box-shadow: var(--shadow-sm);
+        }
+        
+        .leaflet-popup-close-button:hover {
+            background: white;
+            color: var(--gray-800);
+            border-color: var(--gray-400);
+            box-shadow: var(--shadow-md);
+            transform: scale(1.05);
+        }
+        
+        .leaflet-popup-close-button:focus {
+            outline: 2px solid var(--primary-500);
+            outline-offset: 2px;
         }
 
         /* Tree Popup Content */
@@ -907,7 +946,7 @@
 
     <!-- Leaflet JS -->
     <script src="https://unpkg.com/[email protected]/dist/leaflet.js"></script>
-    <script src="/static/map.js?v=4.0.0&t=1754657582"></script>
+    <script src="/static/map.js?v=4.0.1&t=1754657800"></script>
     
     <script>
         // Initialize Granim background animation on page load

static/map.js

@@ -689,26 +689,25 @@ class TreeTrackMap {
     // Tree management methods
     async editTree(treeId) {
         try {
-            const response = await this.authenticatedFetch(`/api/trees/${treeId}`);
-            if (!response) return;
-            
-            if (!response.ok) {
-                throw new Error('Failed to fetch tree data');
+            // Check if user has permission to edit (basic check)
+            if (!this.currentUser) {
+                this.showMessage('Authentication required', 'error');
+                return;
             }
             
             // Store tree ID in localStorage for the form
-            localStorage.setItem('editTreeId', treeId);
+            localStorage.setItem('editTreeId', treeId.toString());
             
-            this.showMessage('Loading tree for editing...', 'success');
+            this.showMessage('Redirecting to form for editing...', 'success');
             
-            // Redirect to form page
+            // Redirect to form page immediately 
             setTimeout(() => {
                 window.location.href = '/';
-            }, 1000);
+            }, 500);
             
         } catch (error) {
-            console.error('Error loading tree for edit:', error);
-            this.showMessage('Error loading tree data: ' + error.message, 'error');
+            console.error('Error setting up tree for edit:', error);
+            this.showMessage('Error preparing tree for editing: ' + error.message, 'error');
         }
     }
     

static/sw.js

@@ -1,5 +1,5 @@
 // TreeTrack Service Worker - PWA and Offline Support
-const VERSION = 1734826222; // Cache busting - updated for location panel button fix & Granim improvements
+const VERSION = 1754657700; // Cache busting - updated for popup close button fix, Granim improvements, and full viewport coverage
 const CACHE_NAME = `treetrack-v${VERSION}`;
 const STATIC_CACHE = `static-v${VERSION}`;
 const API_CACHE = `api-v${VERSION}`;