fix: remove platform check in theme on web

Closes: #89
AppFlowy-IO · Apr 22, 2023 · ef12ee5 · ef12ee5
1 parent 30f231a
commit ef12ee5
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/lib/src/render/style/editor_style.dart b/lib/src/render/style/editor_style.dart
@@ -1,5 +1,6 @@
 import 'dart:io';
 
+import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 
 Iterable<ThemeExtension<dynamic>> get lightEditorStyleExtension => [
@@ -204,7 +205,7 @@ class EditorStyle extends ThemeExtension<EditorStyle> {
   }
 
   static final light = EditorStyle(
-    padding: Platform.isAndroid || Platform.isIOS
+    padding: _isMobile
         ? const EdgeInsets.symmetric(horizontal: 20)
         : const EdgeInsets.symmetric(horizontal: 200),
     backgroundColor: Colors.white,
@@ -258,4 +259,16 @@ class EditorStyle extends ThemeExtension<EditorStyle> {
     popupMenuFGColor: Colors.white,
     popupMenuHoverColor: const Color(0xFF00BCF0),
   );
+
+  static bool get _isMobile {
+    if (kIsWeb) {
+      return false;
+    }
+
+    if (Platform.isAndroid || Platform.isIOS) {
+      return true;
+    }
+
+    return false;
+  }
 }
diff --git a/lib/src/service/shortcut_event/shortcut_event.dart b/lib/src/service/shortcut_event/shortcut_event.dart
@@ -85,6 +85,11 @@ class ShortcutEvent {
     }
     var matched = false;
     if (kIsWeb) {
+      debugPrint("REACHED ONE!");
+      debugPrint("REACHED ONE!!");
+      debugPrint("REACHED ONE!!!");
+      debugPrint("REACHED ONE!!!!");
+      debugPrint("REACHED ONE!!!!!");
       // We shouldn't continue to run the below `else if` code in Web platform, it will throw an `_operatingSystem` exception.
       if (command != null && command.isNotEmpty) {
         this.command = command;