From 21ce1709fe969af1c5858625b59cc2857946c65f Mon Sep 17 00:00:00 2001
From: shenlong-tanwen <139912620+shalong-tanwen@users.noreply.github.com>
Date: Sat, 3 Feb 2024 00:21:53 +0530
Subject: [PATCH] fix(mobile): debounce map layer update

---
 .../lib/extensions/maplibrecontroller_extensions.dart | 11 +++++++++++
 mobile/lib/modules/map/views/map_page.dart            |  5 ++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/mobile/lib/extensions/maplibrecontroller_extensions.dart b/mobile/lib/extensions/maplibrecontroller_extensions.dart
index 0c1e62e3084c2..e01655b3a850b 100644
--- a/mobile/lib/extensions/maplibrecontroller_extensions.dart
+++ b/mobile/lib/extensions/maplibrecontroller_extensions.dart
@@ -1,3 +1,4 @@
+import 'dart:async';
 import 'dart:math';
 
 import 'package:flutter/services.dart';
@@ -6,6 +7,8 @@ import 'package:immich_mobile/modules/map/utils/map_utils.dart';
 import 'package:maplibre_gl/maplibre_gl.dart';
 
 extension MapMarkers on MaplibreMapController {
+  static var _completer = Completer()..complete();
+
   Future<void> addGeoJSONSourceForMarkers(List<MapMarker> markers) async {
     return addSource(
       MapUtils.defaultSourceId,
@@ -16,6 +19,12 @@ extension MapMarkers on MaplibreMapController {
   }
 
   Future<void> reloadAllLayersForMarkers(List<MapMarker> markers) async {
+    // Wait for previous reload to complete
+    if (!_completer.isCompleted) {
+      return _completer.future;
+    }
+    _completer = Completer();
+
     // !! Make sure to remove layers before sources else the native
     // maplibre library would crash when removing the source saying that
     // the source is still in use
@@ -36,6 +45,8 @@ extension MapMarkers on MaplibreMapController {
       MapUtils.defaultHeatMapLayerId,
       MapUtils.defaultHeatMapLayerProperties,
     );
+
+    _completer.complete();
   }
 
   Future<Symbol?> addMarkerAtLatLng(LatLng centre) async {
diff --git a/mobile/lib/modules/map/views/map_page.dart b/mobile/lib/modules/map/views/map_page.dart
index 1882e326165f8..671e501bccc83 100644
--- a/mobile/lib/modules/map/views/map_page.dart
+++ b/mobile/lib/modules/map/views/map_page.dart
@@ -41,6 +41,7 @@ class MapPage extends HookConsumerWidget {
     final bottomSheetStreamController = useStreamController<MapEvent>();
     final selectedMarker = useValueNotifier<_AssetMarkerMeta?>(null);
     final assetsDebouncer = useDebouncer();
+    final layerDebouncer = useDebouncer(interval: const Duration(seconds: 1));
     final isLoading = useProcessingOverlay();
     final scrollController = useScrollController();
     final markerDebouncer =
@@ -77,7 +78,9 @@ class MapPage extends HookConsumerWidget {
     // removes all sources and layers and re-adds them with the updated markers
     Future<void> reloadLayers() async {
       if (mapController.value != null) {
-        mapController.value!.reloadAllLayersForMarkers(markers.value);
+        layerDebouncer.run(
+          () => mapController.value!.reloadAllLayersForMarkers(markers.value),
+        );
       }
     }