From 0df618feee47a4a23ace9ddfe650ce2d8e541fb6 Mon Sep 17 00:00:00 2001
From: Lauritz Tieste <84938977+Lauritz-Tieste@users.noreply.github.com>
Date: Sat, 27 Dec 2025 23:02:42 +0100
Subject: [PATCH] feat: Hide/show controls when zoom state changes (#24784)

feat: hide/show controls based on zoom state in asset viewer
---
 .../widgets/asset_viewer/asset_viewer.page.dart           | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/mobile/lib/presentation/widgets/asset_viewer/asset_viewer.page.dart b/mobile/lib/presentation/widgets/asset_viewer/asset_viewer.page.dart
index d992d243ee..2a7ac9c7fe 100644
--- a/mobile/lib/presentation/widgets/asset_viewer/asset_viewer.page.dart
+++ b/mobile/lib/presentation/widgets/asset_viewer/asset_viewer.page.dart
@@ -92,6 +92,8 @@ class AssetViewer extends ConsumerStatefulWidget {
     if (asset.isVideo || asset.isMotionPhoto) {
       ref.read(videoPlaybackValueProvider.notifier).reset();
       ref.read(videoPlayerControlsProvider.notifier).pause();
+      // Hide controls by default for videos and motion photos
+      ref.read(assetViewerProvider.notifier).setControls(false);
     }
   }
 }
@@ -525,7 +527,13 @@ class _AssetViewerState extends ConsumerState<AssetViewer> {
 
   void _onScaleStateChanged(PhotoViewScaleState scaleState) {
     if (scaleState != PhotoViewScaleState.initial) {
+      ref.read(assetViewerProvider.notifier).setControls(false);
       ref.read(videoPlayerControlsProvider.notifier).pause();
+      return;
+    }
+
+    if (!showingBottomSheet) {
+      ref.read(assetViewerProvider.notifier).setControls(true);
     }
   }