From 9490b6c5b4197d6aa0ac0919e76c3e6341943d62 Mon Sep 17 00:00:00 2001
From: zhanba <c5e1856@gmail.com>
Date: Thu, 22 Feb 2024 18:16:22 +0800
Subject: [PATCH] fix: search view hide not focus libro view

---
 packages/libro-search/src/libro-search-view.tsx | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/packages/libro-search/src/libro-search-view.tsx b/packages/libro-search/src/libro-search-view.tsx
index 54c4c856..4b4b64e8 100644
--- a/packages/libro-search/src/libro-search-view.tsx
+++ b/packages/libro-search/src/libro-search-view.tsx
@@ -142,7 +142,11 @@ export const SearchContent = () => {
                 icon={<EllipsisOutlined />}
                 size="small"
               />
-              <Button onClick={instance.hide} icon={<CloseOutlined />} size="small" />
+              <Button
+                onClick={() => instance.hide()}
+                icon={<CloseOutlined />}
+                size="small"
+              />
             </div>
           </td>
         </tr>
@@ -269,12 +273,13 @@ export class LibroSearchView extends BaseView {
       this.toDispose.push(
         this.libro.model.onCellViewChanged(() => this.onCellsChanged()),
       );
+      // search view 失去焦点时依然可以通过esc隐藏search
       this.toDispose.push(
         this.libro.model.onCommandModeChanged((mode) => {
           if (mode) {
             setTimeout(() => {
               if (this.hasFocus === false) {
-                this.hide();
+                this.hide(false);
               }
             }, 0);
           }
@@ -306,14 +311,16 @@ export class LibroSearchView extends BaseView {
     this.initialQuery();
     this.findInputRef?.current?.focus();
   };
-  hide = () => {
+  hide = (focus = true) => {
     this.searchVisible = false;
     this.contextKey.enableCommandMode();
     this.searchProvider?.endQuery();
     if (this.searchProvider) {
       this.searchProvider.replaceMode = false;
     }
-    this.libro?.focus();
+    if (focus) {
+      this.libro?.focus();
+    }
   };
 
   onFocus = () => {