feat(performance): dispose every widget on window hidden and re-initi…

…alize when shown
KRTirtho · Mar 23, 2023 · 1ed5452 · 1ed5452
1 parent 6dd24d3
commit 1ed5452
Show file tree

Hide file tree

Showing 4 changed files with 207 additions and 17 deletions.
diff --git a/lib/collections/shortcuts_registry.dart b/lib/collections/shortcuts_registry.dart
@@ -20,22 +20,28 @@ class ShortcutsRegistry {
   final Map<ShortcutAction, VoidCallback> _shortcuts = {
     ShortcutAction.openFlemozi: () async {
       if (await windowManager.isVisible() && !await windowManager.isFocused()) {
-        if (kIsLinux) await windowManager.setAlwaysOnTop(true);
+        if (kIsLinux) {
+          await windowManager.setAlwaysOnTop(true);
+        }
         await windowManager.focus();
+        if (kIsLinux) await windowManager.grabKeyboard();
         if (kIsLinux) {
           Future.delayed(const Duration(milliseconds: 100), () async {
             await windowManager.setAlwaysOnTop(false);
             await windowManager.focus();
+            await windowManager.grabKeyboard();
           });
         }
       } else {
         if (kIsLinux) await windowManager.setAlwaysOnTop(true);
         await windowManager.show();
         await windowManager.focus();
+        if (kIsLinux) await windowManager.grabKeyboard();
         if (kIsLinux) {
           Future.delayed(const Duration(milliseconds: 100), () async {
             await windowManager.setAlwaysOnTop(false);
             await windowManager.focus();
+            await windowManager.grabKeyboard();
           });
         }
       }

diff --git a/lib/hooks/use_window_listeners.dart b/lib/hooks/use_window_listeners.dart
@@ -0,0 +1,164 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_hooks/flutter_hooks.dart';
+import 'package:window_manager/window_manager.dart';
+
+void useWindowListeners({
+  VoidCallback? onWindowClose,
+  VoidCallback? onWindowFocus,
+  VoidCallback? onWindowBlur,
+  VoidCallback? onWindowMaximize,
+  VoidCallback? onWindowUnmaximize,
+  VoidCallback? onWindowMinimize,
+  VoidCallback? onWindowRestore,
+  VoidCallback? onWindowResize,
+  VoidCallback? onWindowResized,
+  VoidCallback? onWindowMove,
+  VoidCallback? onWindowMoved,
+  VoidCallback? onWindowEnterFullScreen,
+  VoidCallback? onWindowLeaveFullScreen,
+  ValueChanged<String>? onWindowEvent,
+}) {
+  return use(
+    UseWindowListeners(
+      onWindowClose: onWindowClose,
+      onWindowFocus: onWindowFocus,
+      onWindowBlur: onWindowBlur,
+      onWindowMaximize: onWindowMaximize,
+      onWindowUnmaximize: onWindowUnmaximize,
+      onWindowMinimize: onWindowMinimize,
+      onWindowRestore: onWindowRestore,
+      onWindowResize: onWindowResize,
+      onWindowResized: onWindowResized,
+      onWindowMove: onWindowMove,
+      onWindowMoved: onWindowMoved,
+      onWindowEnterFullScreen: onWindowEnterFullScreen,
+      onWindowLeaveFullScreen: onWindowLeaveFullScreen,
+      onWindowEvent: onWindowEvent,
+    ),
+  );
+}
+
+class UseWindowListeners extends Hook {
+  final VoidCallback? onWindowClose;
+  final VoidCallback? onWindowFocus;
+  final VoidCallback? onWindowBlur;
+  final VoidCallback? onWindowMaximize;
+  final VoidCallback? onWindowUnmaximize;
+  final VoidCallback? onWindowMinimize;
+  final VoidCallback? onWindowRestore;
+  final VoidCallback? onWindowResize;
+  final VoidCallback? onWindowResized;
+  final VoidCallback? onWindowMove;
+  final VoidCallback? onWindowMoved;
+  final VoidCallback? onWindowEnterFullScreen;
+  final VoidCallback? onWindowLeaveFullScreen;
+  final ValueChanged<String>? onWindowEvent;
+
+  const UseWindowListeners({
+    this.onWindowClose,
+    this.onWindowFocus,
+    this.onWindowBlur,
+    this.onWindowMaximize,
+    this.onWindowUnmaximize,
+    this.onWindowMinimize,
+    this.onWindowRestore,
+    this.onWindowResize,
+    this.onWindowResized,
+    this.onWindowMove,
+    this.onWindowMoved,
+    this.onWindowEnterFullScreen,
+    this.onWindowLeaveFullScreen,
+    this.onWindowEvent,
+  });
+
+  @override
+  _UseWindowListenersState createState() => _UseWindowListenersState();
+}
+
+class _UseWindowListenersState extends HookState<void, UseWindowListeners>
+    implements WindowListener {
+  @override
+  void initHook() {
+    super.initHook();
+    WindowManager.instance.addListener(this);
+  }
+
+  @override
+  void dispose() {
+    WindowManager.instance.removeListener(this);
+    super.dispose();
+  }
+
+  @override
+  void build(BuildContext context) {}
+
+  @override
+  void onWindowBlur() {
+    hook.onWindowBlur?.call();
+  }
+
+  @override
+  void onWindowClose() {
+    hook.onWindowClose?.call();
+  }
+
+  @override
+  void onWindowEnterFullScreen() {
+    hook.onWindowEnterFullScreen?.call();
+  }
+
+  @override
+  void onWindowEvent(String eventName) {
+    hook.onWindowEvent?.call(eventName);
+  }
+
+  @override
+  void onWindowFocus() {
+    hook.onWindowFocus?.call();
+  }
+
+  @override
+  void onWindowLeaveFullScreen() {
+    hook.onWindowLeaveFullScreen?.call();
+  }
+
+  @override
+  void onWindowMaximize() {
+    hook.onWindowMaximize?.call();
+  }
+
+  @override
+  void onWindowMinimize() {
+    hook.onWindowMinimize?.call();
+  }
+
+  @override
+  void onWindowMove() {
+    hook.onWindowMove?.call();
+  }
+
+  @override
+  void onWindowMoved() {
+    hook.onWindowMoved?.call();
+  }
+
+  @override
+  void onWindowResize() {
+    hook.onWindowResize?.call();
+  }
+
+  @override
+  void onWindowResized() {
+    hook.onWindowResized?.call();
+  }
+
+  @override
+  void onWindowRestore() {
+    hook.onWindowRestore?.call();
+  }
+
+  @override
+  void onWindowUnmaximize() {
+    hook.onWindowUnmaximize?.call();
+  }
+}
diff --git a/lib/intents/close_window.dart b/lib/intents/close_window.dart
@@ -1,3 +1,4 @@
+import 'package:flemozi/utils/platform.dart';
 import 'package:flutter/material.dart';
 import 'package:window_manager/window_manager.dart';
 
@@ -8,6 +9,7 @@ class CloseWindowIntent extends Intent {
 class CloseWindowAction extends Action<CloseWindowIntent> {
   @override
   void invoke(CloseWindowIntent intent) async {
+    if (kIsLinux) await windowManager.ungrabKeyboard();
     await windowManager.hide();
   }
 }
diff --git a/lib/pages/root.dart b/lib/pages/root.dart
@@ -2,6 +2,7 @@ import 'package:flemozi/components/root/emoji.dart';
 import 'package:flemozi/components/root/gif.dart';
 import 'package:flemozi/components/ui/top_bar.dart';
 import 'package:flemozi/components/ui/vertical_tabs.dart';
+import 'package:flemozi/hooks/use_window_listeners.dart';
 import 'package:flemozi/intents/close_window.dart';
 import 'package:flemozi/pages/settings/settings.dart';
 import 'package:flutter/material.dart';
@@ -13,6 +14,21 @@ class RootPage extends HookWidget {
 
   @override
   Widget build(BuildContext context) {
+    final windowIsShowing = useState(true);
+
+    useWindowListeners(onWindowEvent: (String eventName) {
+      switch (eventName) {
+        case "hide":
+          windowIsShowing.value = false;
+          break;
+        case "show":
+          windowIsShowing.value = true;
+          break;
+        default:
+          break;
+      }
+    });
+
     return CallbackShortcuts(
       bindings: {
         LogicalKeySet(
@@ -28,22 +44,24 @@ class RootPage extends HookWidget {
       },
       child: Scaffold(
         appBar: const TopBar(),
-        body: VerticalTabs(
-          tabs: const [
-            Tooltip(
-              message: 'Emoji',
-              child: Icon(Icons.emoji_emotions),
-            ),
-            Tooltip(
-              message: 'GIFs',
-              child: Icon(Icons.gif_rounded),
-            ),
-          ],
-          children: const [
-            Emoji(),
-            Gif(),
-          ],
-        ),
+        body: windowIsShowing.value
+            ? VerticalTabs(
+                tabs: const [
+                  Tooltip(
+                    message: 'Emoji',
+                    child: Icon(Icons.emoji_emotions),
+                  ),
+                  Tooltip(
+                    message: 'GIFs',
+                    child: Icon(Icons.gif_rounded),
+                  ),
+                ],
+                children: const [
+                  Emoji(),
+                  Gif(),
+                ],
+              )
+            : const SizedBox.shrink(),
       ),
     );
   }