chore: Follow up image viewer

krille-chan · Jan 6, 2025 · b2131e6 · b2131e6
1 parent f7bf273
commit b2131e6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 10 deletions.
diff --git a/assets/l10n/intl_en.arb b/assets/l10n/intl_en.arb
@@ -2820,5 +2820,6 @@
   "pleaseFillOut": "Please fill out",
   "invalidUrl": "Invalid url",
   "addLink": "Add link",
-  "unableToJoinChat": "Unable to join chat. Maybe the other party has already closed the conversation."
+  "unableToJoinChat": "Unable to join chat. Maybe the other party has already closed the conversation.",
+  "previous": "Previous"
 }
diff --git a/lib/pages/image_viewer/image_viewer_view.dart b/lib/pages/image_viewer/image_viewer_view.dart
@@ -57,9 +57,7 @@ class ImageViewerView extends StatelessWidget {
                 padding: const EdgeInsets.only(right: 8.0),
                 child: Builder(
                   builder: (context) => IconButton(
-                    style: IconButton.styleFrom(
-                      backgroundColor: Colors.black.withAlpha(128),
-                    ),
+                    style: iconButtonStyle,
                     onPressed: () => controller.shareFileAction(context),
                     tooltip: L10n.of(context).share,
                     color: Colors.white,
@@ -82,12 +80,16 @@ class ImageViewerView extends StatelessWidget {
                   child: Center(
                     child: Hero(
                       tag: controller.allEvents[i].eventId,
-                      child: MxcImage(
-                        key: ValueKey(controller.allEvents[i].eventId),
-                        event: controller.allEvents[i],
-                        fit: BoxFit.contain,
-                        isThumbnail: false,
-                        animated: true,
+                      child: GestureDetector(
+                        // Ignore taps to not go back here:
+                        onTap: () {},
+                        child: MxcImage(
+                          key: ValueKey(controller.allEvents[i].eventId),
+                          event: controller.allEvents[i],
+                          fit: BoxFit.contain,
+                          isThumbnail: false,
+                          animated: true,
+                        ),
                       ),
                     ),
                   ),
@@ -100,6 +102,7 @@ class ImageViewerView extends StatelessWidget {
                     padding: const EdgeInsets.all(12.0),
                     child: IconButton(
                       style: iconButtonStyle,
+                      tooltip: L10n.of(context).previous,
                       icon: const Icon(Icons.chevron_left_outlined),
                       onPressed: () => controller.pageController.previousPage(
                         duration: FluffyThemes.animationDuration,
@@ -115,6 +118,7 @@ class ImageViewerView extends StatelessWidget {
                     padding: const EdgeInsets.all(12.0),
                     child: IconButton(
                       style: iconButtonStyle,
+                      tooltip: L10n.of(context).next,
                       icon: const Icon(Icons.chevron_right_outlined),
                       onPressed: () => controller.pageController.nextPage(
                         duration: FluffyThemes.animationDuration,