fix: preserve 0.0 coordinates and improve memories UX

Author: dehydrated-bear

backend/app/database/images.py

@@ -889,8 +889,8 @@ def db_get_all_images_for_memories() -> List[dict]:
                     "metadata": image_util_parse_metadata(row[4]),
                     "isTagged": bool(row[5]),
                     "isFavourite": bool(row[6]),
-                    "latitude": row[7] if row[7] else None,  # Can be None
-                    "longitude": row[8] if row[8] else None,  # Can be None
+                    "latitude": row[7] if row[7] is not None else None,  # Can be None
+                    "longitude": row[8] if row[8] is not None else None,  # Can be None
                     "captured_at": row[9] if row[9] else None,
                     "tags": row[10].split(",") if row[10] else None,
                 }

frontend/src/components/Memories/MemoriesPage.tsx

@@ -23,7 +23,6 @@ import {
   selectAllMemories,
   selectMemoriesLoading,
   selectMemoriesError,
-  selectTotalMemoryCount,
 } from '@/store/slices/memoriesSlice';
 import { MemoryCard } from './MemoryCard';
 import { FeaturedMemoryCard } from './FeaturedMemoryCard';
@@ -148,7 +147,6 @@ export const MemoriesPage: React.FC = () => {
   const allMemories = useAppSelector(selectAllMemories);
   const loading = useAppSelector(selectMemoriesLoading);
   const error = useAppSelector(selectMemoriesError);
-  const totalCount = useAppSelector(selectTotalMemoryCount);
 
   // Simple filter state: 'all' | 'location' | 'date'
   const [filter, setFilter] = useState<'all' | 'location' | 'date'>('all');
@@ -158,6 +156,7 @@ export const MemoriesPage: React.FC = () => {
     memories.filter((m) => m.image_count >= 2);
 
   // Calculate counts (only memories with 2+ images)
+  const totalCount = memoriesWithMultipleImages(allMemories).length;
   const locationCount = memoriesWithMultipleImages(allMemories).filter(
     (m) => m.center_lat != null && m.center_lon != null,
   ).length;

frontend/src/components/Memories/MemoryDetail.tsx

@@ -114,24 +114,18 @@ export const MemoryDetail = () => {
         file_location: img.path,
         file_size: 0,
         item_type: 'image' as const,
-        latitude: img.latitude || undefined,
-        longitude: img.longitude || undefined,
+        latitude: img.latitude ?? undefined,
+        longitude: img.longitude ?? undefined,
       },
     }));
 
-    dispatch(setImages(formattedImages));
-    dispatch(hideLoader());
+    // Defer setImages and hideLoader to next tick so loader becomes visible
+    requestAnimationFrame(() => {
+      dispatch(setImages(formattedImages));
+      dispatch(hideLoader());
+    });
   }, [memory, dispatch]);
 
-  // Refetch memory data when returning to the page to get updated favorite status
-  useEffect(() => {
-    // This will cause the component to re-render with fresh data from Redux
-    // when the user toggles a favorite and the memory data is refetched
-    return () => {
-      // Cleanup if needed
-    };
-  }, []);
-
   // If memory not found, show error
   if (!memory) {
     return (

frontend/src/store/slices/memoriesSlice.ts

@@ -162,18 +162,13 @@ export const fetchAllMemoriesData = createAsyncThunk<
   void,
   void,
   { rejectValue: string }
->('memories/fetchAllData', async (_, { dispatch, rejectWithValue }) => {
-  try {
-    await Promise.all([
-      dispatch(fetchOnThisDay()),
-      dispatch(fetchRecentMemories(30)),
-      dispatch(fetchYearMemories(365)),
-      dispatch(fetchAllMemories()),
-    ]);
-  } catch (error) {
-    const apiError = error as ApiError;
-    return rejectWithValue(apiError.message);
-  }
+>('memories/fetchAllData', async (_, { dispatch }) => {
+  await Promise.all([
+    dispatch(fetchOnThisDay()),
+    dispatch(fetchRecentMemories(30)),
+    dispatch(fetchYearMemories(365)),
+    dispatch(fetchAllMemories()),
+  ]);
 });
 
 // ============================================================================