From fac80d98b9846bbbf7a67f0fa7b379def2ff3074 Mon Sep 17 00:00:00 2001
From: Thibault Deckers <thibault.deckers@gmail.com>
Date: Sat, 10 Aug 2019 16:08:21 +0900
Subject: [PATCH] fullscreen: context menu

---
 lib/widgets/fullscreen/image_page.dart  | 36 ++++++++++++++++++-------
 lib/widgets/fullscreen/overlay_top.dart | 25 +++++++++++------
 2 files changed, 43 insertions(+), 18 deletions(-)

diff --git a/lib/widgets/fullscreen/image_page.dart b/lib/widgets/fullscreen/image_page.dart
index 63e288e05..4c1ae145f 100644
--- a/lib/widgets/fullscreen/image_page.dart
+++ b/lib/widgets/fullscreen/image_page.dart
@@ -1,6 +1,7 @@
 import 'dart:io';
 import 'dart:math';
 
+import 'package:aves/model/android_app_service.dart';
 import 'package:aves/model/image_entry.dart';
 import 'package:aves/widgets/fullscreen/info/info_page.dart';
 import 'package:aves/widgets/fullscreen/overlay_bottom.dart';
@@ -79,6 +80,7 @@ class FullscreenPageState extends State<FullscreenPage> with SingleTickerProvide
 
   @override
   Widget build(BuildContext context) {
+    final entry = entries[_currentHorizontalPage];
     return WillPopScope(
       onWillPop: () {
         Screen.keepOn(false);
@@ -104,18 +106,10 @@ class FullscreenPageState extends State<FullscreenPage> with SingleTickerProvide
                 ),
                 NotificationListener(
                   onNotification: (notification) {
-                    if (notification is BackUpNotification) {
-                      _verticalPager.animateToPage(
-                        0,
-                        duration: const Duration(milliseconds: 350),
-                        curve: Curves.easeInOut,
-                      );
-                    }
+                    if (notification is BackUpNotification) goToVerticalPage(0);
                     return false;
                   },
-                  child: InfoPage(
-                    entry: entries[_currentHorizontalPage],
-                  ),
+                  child: InfoPage(entry: entry),
                 ),
               ],
             ),
@@ -126,6 +120,7 @@ class FullscreenPageState extends State<FullscreenPage> with SingleTickerProvide
                 scale: _topOverlayScale,
                 viewInsets: _frozenViewInsets,
                 viewPadding: _frozenViewPadding,
+                onActionSelected: (action) => onActionSelected(entry, action),
               ),
               Positioned(
                 bottom: 0,
@@ -175,6 +170,14 @@ class FullscreenPageState extends State<FullscreenPage> with SingleTickerProvide
     );
   }
 
+  goToVerticalPage(int page) {
+    return _verticalPager.animateToPage(
+      page,
+      duration: Duration(milliseconds: 350),
+      curve: Curves.easeInOut,
+    );
+  }
+
   onOverlayVisibleChange() async {
     if (_overlayVisible.value) {
       SystemChrome.setEnabledSystemUIOverlays(SystemUiOverlay.values);
@@ -189,8 +192,21 @@ class FullscreenPageState extends State<FullscreenPage> with SingleTickerProvide
       _frozenViewPadding = null;
     }
   }
+
+  onActionSelected(ImageEntry entry, FullscreenAction action) {
+    switch (action) {
+      case FullscreenAction.info:
+        goToVerticalPage(1);
+        break;
+      case FullscreenAction.share:
+        AndroidAppService.share(entry.uri, entry.mimeType);
+        break;
+    }
+  }
 }
 
+enum FullscreenAction { info, share }
+
 class ImagePage extends StatefulWidget {
   final List<ImageEntry> entries;
   final PageController pageController;
diff --git a/lib/widgets/fullscreen/overlay_top.dart b/lib/widgets/fullscreen/overlay_top.dart
index 05ccfd091..a16e16ac4 100644
--- a/lib/widgets/fullscreen/overlay_top.dart
+++ b/lib/widgets/fullscreen/overlay_top.dart
@@ -1,6 +1,6 @@
-import 'package:aves/model/android_app_service.dart';
 import 'package:aves/model/image_entry.dart';
 import 'package:aves/widgets/common/blurred.dart';
+import 'package:aves/widgets/fullscreen/image_page.dart';
 import 'package:flutter/material.dart';
 
 class FullscreenTopOverlay extends StatelessWidget {
@@ -8,6 +8,7 @@ class FullscreenTopOverlay extends StatelessWidget {
   final int index;
   final Animation<double> scale;
   final EdgeInsets viewInsets, viewPadding;
+  final Function(FullscreenAction value) onActionSelected;
 
   ImageEntry get entry => entries[index];
 
@@ -18,6 +19,7 @@ class FullscreenTopOverlay extends StatelessWidget {
     this.scale,
     this.viewInsets,
     this.viewPadding,
+    this.onActionSelected,
   }) : super(key: key);
 
   @override
@@ -37,21 +39,28 @@ class FullscreenTopOverlay extends StatelessWidget {
               scale: scale,
               child: IconButton(
                 icon: Icon(Icons.share),
-                onPressed: share,
+                onPressed: () => onActionSelected?.call(FullscreenAction.share),
                 tooltip: 'Share',
               ),
             ),
+            SizedBox(width: 8),
+            OverlayButton(
+              scale: scale,
+              child: PopupMenuButton<FullscreenAction>(
+                itemBuilder: (context) => [
+                  PopupMenuItem(
+                    value: FullscreenAction.info,
+                    child: Text("Info"),
+                  ),
+                ],
+                onSelected: onActionSelected,
+              ),
+            ),
           ],
         ),
       ),
     );
   }
-
-  delete() {}
-
-  share() {
-    AndroidAppService.share(entry.uri, entry.mimeType);
-  }
 }
 
 class OverlayButton extends StatelessWidget {