fix: the toolbar shouldn’t over the search panel

LucasXu0 · Jan 10, 2025 · 186f205 · 186f205
1 parent 823b744
commit 186f205
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart b/frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart
@@ -4,6 +4,7 @@ import 'dart:io';
 import 'package:appflowy/generated/flowy_svgs.g.dart';
 import 'package:appflowy/generated/locale_keys.g.dart';
 import 'package:appflowy/plugins/blank/blank.dart';
+import 'package:appflowy/plugins/document/presentation/editor_notification.dart';
 import 'package:appflowy/shared/feature_flags.dart';
 import 'package:appflowy/shared/loading.dart';
 import 'package:appflowy/startup/startup.dart';
@@ -469,7 +470,11 @@ class _SidebarSearchButton extends StatelessWidget {
         ],
       ),
       child: FlowyButton(
-        onTap: () => CommandPalette.of(context).toggle(),
+        onTap: () {
+          // exit editing mode when doing search to avoid the toolbar showing up
+          EditorNotification.exitEditing().post();
+          CommandPalette.of(context).toggle();
+        },
         leftIcon: const FlowySvg(FlowySvgs.search_s),
         iconPadding: 12.0,
         margin: const EdgeInsets.only(left: 8.0),