Stop having findInPage and Panel opened at same time

We shouldn't let user feel that we can search content in the library page through findInPage. So we close findInPage when we open the Library, and we close the panel when we open findInPage in the hamburger menu. We shall pay attention to this as well in #881 once this is merged Signed-off-by: Songlin Jiang <[email protected]>
Igalia · Sep 6, 2023 · 7049cbc · 7049cbc
1 parent 118ba3f
commit 7049cbc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/app/src/common/shared/com/igalia/wolvic/ui/widgets/NavigationBarWidget.java b/app/src/common/shared/com/igalia/wolvic/ui/widgets/NavigationBarWidget.java
@@ -1285,6 +1285,7 @@ public void onSendTab() {
             @Override
             public void onFindInPage() {
                 hideMenu();
+                mAttachedWindow.hidePanel();
 
                 mViewModel.setIsFindInPage(true);
             }

diff --git a/app/src/common/shared/com/igalia/wolvic/ui/widgets/WindowWidget.java b/app/src/common/shared/com/igalia/wolvic/ui/widgets/WindowWidget.java
@@ -519,6 +519,7 @@ private void showPanel(@Windows.PanelType int panelType, boolean switchSurface)
                 setView(mLibrary, switchSurface);
                 mLibrary.selectPanel(panelType);
                 mLibrary.onShow();
+                mViewModel.setIsFindInPage(false);
                 mViewModel.setIsPanelVisible(true);
                 if (mRestoreFirstPaint == null && !isFirstPaintReady() && (mFirstDrawCallback != null) && (mSurface != null)) {
                     final Runnable firstDrawCallback = mFirstDrawCallback;