From 366ab1d173f01e003e9e7b22e0c09e57eb6022a2 Mon Sep 17 00:00:00 2001
From: ShaopengLin <shaopeng.lin@mail.utoronto.ca>
Date: Fri, 15 Mar 2024 02:26:53 -0400
Subject: [PATCH] SearchBar shortcuts work now

Refactored shortcut to search bar and removed redundant code. Fix #967
---
 src/kiwixapp.cpp   |  3 +--
 src/mainwindow.cpp | 12 ------------
 src/mainwindow.h   |  1 -
 src/searchbar.cpp  | 10 +++++++++-
 4 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/kiwixapp.cpp b/src/kiwixapp.cpp
index 45650ddc..eb7ff09f 100644
--- a/src/kiwixapp.cpp
+++ b/src/kiwixapp.cpp
@@ -439,8 +439,7 @@ void KiwixApp::createActions()
     HIDE_ACTION(SavePageAsAction);
     */
 
-    CREATE_ACTION_SHORTCUT(SearchArticleAction, gt("search-article"), QKeySequence(Qt::CTRL | Qt::Key_L));
-    HIDE_ACTION(SearchArticleAction);
+    CREATE_ACTION_SHORTCUTS(SearchArticleAction, gt("search-article"), QList<QKeySequence>({QKeySequence(Qt::Key_F6), QKeySequence(Qt::CTRL | Qt::Key_L), QKeySequence(Qt::ALT | Qt::Key_D)}));
 
     CREATE_ACTION_SHORTCUT(SearchLibraryAction, gt("search-in-library"), QKeySequence(Qt::CTRL | Qt::SHIFT | Qt::Key_R));
     HIDE_ACTION(SearchLibraryAction);
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index ab70c410..df3b8635 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -148,18 +148,6 @@ void MainWindow::when_libraryPageDisplayed(bool showed)
     }
 }
 
-void MainWindow::keyPressEvent(QKeyEvent *event)
-{
-    auto key = event->key();
-    auto modifier = event->modifiers();
-    if (key == Qt::Key_F6 ||
-        (key == Qt::Key_L && modifier == Qt::ControlModifier) ||
-        (key == Qt::Key_D && modifier == Qt::AltModifier)) {
-        getTopWidget()->getSearchBar().selectAll();
-        getTopWidget()->getSearchBar().setFocus();
-    }
-    return QWidget::keyPressEvent(event);
-}
 TabBar* MainWindow::getTabBar()
 {
     return mp_ui->tabBar;
diff --git a/src/mainwindow.h b/src/mainwindow.h
index 701f4255..41b2fe92 100644
--- a/src/mainwindow.h
+++ b/src/mainwindow.h
@@ -27,7 +27,6 @@ class MainWindow : public QMainWindow
     QWidget getMainView();
 
 protected:
-    void keyPressEvent(QKeyEvent *event) override;
     bool eventFilter(QObject* object, QEvent* event) override;
 
 private slots:
diff --git a/src/searchbar.cpp b/src/searchbar.cpp
index 3c3628af..2f0d0211 100644
--- a/src/searchbar.cpp
+++ b/src/searchbar.cpp
@@ -94,6 +94,13 @@ SearchBar::SearchBar(QWidget *parent) :
     connect(this, &QLineEdit::returnPressed, this, [=]() {
         m_returnPressed = true;
     });
+    
+    auto app = KiwixApp::instance();
+    connect(app->getAction(KiwixApp::SearchArticleAction), &QAction::triggered,
+            this, [=]() {
+                this->selectAll();
+                this->setFocus(Qt::ShortcutFocusReason);
+            });
 }
 
 void SearchBar::hideSuggestions()
@@ -129,7 +136,8 @@ void SearchBar::focusInEvent( QFocusEvent* event)
         clear();
     }
     if (event->reason() == Qt::ActiveWindowFocusReason ||
-        event->reason() == Qt::MouseFocusReason) {
+        event->reason() == Qt::MouseFocusReason ||
+        event->reason() == Qt::ShortcutFocusReason) {
         connect(&m_completer, QOverload<const QModelIndex &>::of(&QCompleter::activated),
         this, QOverload<const QModelIndex &>::of(&SearchBar::openCompletion));
     }