Add a small delay before processing the key input of search boxes

PR #20465.
Closes #20025.
Closes #20235.

src/gui/addnewtorrentdialog.cpp

@@ -33,6 +33,7 @@
 #include <functional>
 
 #include <QAction>
+#include <QByteArray>
 #include <QDateTime>
 #include <QDebug>
 #include <QDir>
@@ -41,6 +42,7 @@
 #include <QPushButton>
 #include <QScreen>
 #include <QShortcut>
+#include <QSize>
 #include <QString>
 #include <QUrl>
 #include <QVector>

src/gui/addnewtorrentdialog.h

@@ -39,6 +39,9 @@
 #include "base/path.h"
 #include "base/settingvalue.h"
 
+class LineEdit;
+class TorrentFileGuard;
+
 namespace BitTorrent
 {
     class InfoHash;
@@ -54,9 +57,6 @@ namespace Ui
     class AddNewTorrentDialog;
 }
 
-class LineEdit;
-class TorrentFileGuard;
-
 class AddNewTorrentDialog final : public QDialog
 {
     Q_OBJECT

src/gui/lineedit.cpp

@@ -29,20 +29,41 @@
 
 #include "lineedit.h"
 
+#include <chrono>
+
 #include <QAction>
 #include <QKeyEvent>
+#include <QTimer>
 
 #include "base/global.h"
 #include "uithememanager.h"
 
+using namespace std::chrono_literals;
+
+namespace
+{
+    const std::chrono::milliseconds FILTER_INPUT_DELAY {400};
+}
+
 LineEdit::LineEdit(QWidget *parent)
     : QLineEdit(parent)
+    , m_delayedTextChangedTimer {new QTimer(this)}
 {
     auto *action = new QAction(UIThemeManager::instance()->getIcon(u"edit-find"_s), QString(), this);
     addAction(action, QLineEdit::LeadingPosition);
 
     setClearButtonEnabled(true);
     setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
+
+    m_delayedTextChangedTimer->setSingleShot(true);
+    connect(m_delayedTextChangedTimer, &QTimer::timeout, this, [this]
+    {
+        emit textChanged(text());
+    });
+    connect(this, &QLineEdit::textChanged, this, [this]
+    {
+        m_delayedTextChangedTimer->start(FILTER_INPUT_DELAY);
+    });
 }
 
 void LineEdit::keyPressEvent(QKeyEvent *event)

src/gui/lineedit.h

@@ -31,6 +31,9 @@
 
 #include <QLineEdit>
 
+class QKeyEvent;
+class QTimer;
+
 class LineEdit final : public QLineEdit
 {
     Q_OBJECT
@@ -39,6 +42,11 @@ class LineEdit final : public QLineEdit
 public:
     explicit LineEdit(QWidget *parent = nullptr);
 
+signals:
+    void textChanged(const QString &text);
+
 private:
     void keyPressEvent(QKeyEvent *event) override;
+
+    QTimer *m_delayedTextChangedTimer = nullptr;
 };

src/gui/mainwindow.cpp

@@ -57,6 +57,7 @@
 #include <QShortcut>
 #include <QSplitter>
 #include <QStatusBar>
+#include <QString>
 #include <QtGlobal>
 #include <QTimer>
 

src/gui/mainwindow.h

@@ -42,6 +42,7 @@ class QCloseEvent;
 class QComboBox;
 class QFileSystemWatcher;
 class QSplitter;
+class QString;
 class QTabWidget;
 class QTimer;
 

src/gui/search/searchjobwidget.cpp

@@ -128,9 +128,9 @@ SearchJobWidget::SearchJobWidget(SearchHandler *searchHandler, QWidget *parent)
     m_lineEditSearchResultsFilter->setPlaceholderText(tr("Filter search results..."));
     m_lineEditSearchResultsFilter->setContextMenuPolicy(Qt::CustomContextMenu);
     connect(m_lineEditSearchResultsFilter, &QWidget::customContextMenuRequested, this, &SearchJobWidget::showFilterContextMenu);
+    connect(m_lineEditSearchResultsFilter, &LineEdit::textChanged, this, &SearchJobWidget::filterSearchResults);
     m_ui->horizontalLayout->insertWidget(0, m_lineEditSearchResultsFilter);
 
-    connect(m_lineEditSearchResultsFilter, &LineEdit::textChanged, this, &SearchJobWidget::filterSearchResults);
     connect(m_ui->filterMode, qOverload<int>(&QComboBox::currentIndexChanged)
             , this, &SearchJobWidget::updateFilter);
     connect(m_ui->minSeeds, &QAbstractSpinBox::editingFinished, this, &SearchJobWidget::updateFilter);

src/webui/www/private/scripts/client.js

@@ -658,6 +658,8 @@ window.addEvent('load', function() {
                 $('error_div').set('html', '');
                 if (response) {
                     clearTimeout(torrentsFilterInputTimer);
+                    torrentsFilterInputTimer = -1;
+
                     let torrentsTableSelectedRows;
                     let update_categories = false;
                     let updateTags = false;
@@ -1357,18 +1359,14 @@ window.addEvent('load', function() {
         $('torrentFilesFilterToolbar').addClass("invisible");
     };
 
-    let prevTorrentsFilterValue;
-    let torrentsFilterInputTimer = null;
     // listen for changes to torrentsFilterInput
-    $('torrentsFilterInput').addEvent('input', function() {
+    let torrentsFilterInputTimer = -1;
-        const value = $('torrentsFilterInput').get("value");
+    $('torrentsFilterInput').addEvent('input', () => {
-        if (value !== prevTorrentsFilterValue) {
+        clearTimeout(torrentsFilterInputTimer);
-            prevTorrentsFilterValue = value;
+        torrentsFilterInputTimer = setTimeout(() => {
-            clearTimeout(torrentsFilterInputTimer);
+            torrentsFilterInputTimer = -1;
-            torrentsFilterInputTimer = setTimeout(function() {
+            torrentsTable.updateTable();
-                torrentsTable.updateTable(false);
+        }, window.qBittorrent.Misc.FILTER_INPUT_DELAY);
-            }, 400);
-        }
     });
 
     $('transfersTabLink').addEvent('click', showTransfersTab);

src/webui/www/private/scripts/contextmenu.js

@@ -175,12 +175,14 @@ window.qBittorrent.ContextMenu = (function() {
 
                 const touchstartEvent = e;
                 this.touchstartTimer = setTimeout(function() {
+                    this.touchstartTimer = -1;
                     this.triggerMenu(touchstartEvent, elem);
                 }.bind(this), this.options.touchTimer);
             }.bind(this));
             elem.addEvent('touchend', function(e) {
                 e.preventDefault();
                 clearTimeout(this.touchstartTimer);
+                this.touchstartTimer = -1;
             }.bind(this));
         },
 

src/webui/www/private/scripts/dynamicTable.js

@@ -701,10 +701,7 @@ window.qBittorrent.DynamicTable = (function() {
             return null;
         },
 
-        updateTable: function(fullUpdate) {
+        updateTable: function(fullUpdate = false) {
-            if (fullUpdate === undefined)
-                fullUpdate = false;
-
             const rows = this.getFilteredAndSortedRows();
 
             for (let i = 0; i < this.selectedRows.length; ++i)

src/webui/www/private/scripts/misc.js

@@ -46,6 +46,8 @@ window.qBittorrent.Misc = (function() {
             toFixedPointString: toFixedPointString,
             containsAllTerms: containsAllTerms,
             sleep: sleep,
+            // variables
+            FILTER_INPUT_DELAY: 400,
             MAX_ETA: 8640000
         };
     };

src/webui/www/private/scripts/prop-files.js

@@ -366,6 +366,7 @@ window.qBittorrent.PropFiles = (function() {
             },
             onSuccess: function(files) {
                 clearTimeout(torrentFilesFilterInputTimer);
+                torrentFilesFilterInputTimer = -1;
 
                 if (files.length === 0) {
                     torrentFilesTable.clear();
@@ -640,26 +641,27 @@ window.qBittorrent.PropFiles = (function() {
     if (torrentFilesTable.getSortedColumn() === null)
         torrentFilesTable.setSortedColumn('name');
 
-    let prevTorrentFilesFilterValue;
-    let torrentFilesFilterInputTimer = null;
     // listen for changes to torrentFilesFilterInput
-    $('torrentFilesFilterInput').addEvent('input', function() {
+    let torrentFilesFilterInputTimer = -1;
-        const value = $('torrentFilesFilterInput').get("value");
+    $('torrentFilesFilterInput').addEvent('input', () => {
-        if (value !== prevTorrentFilesFilterValue) {
+        clearTimeout(torrentFilesFilterInputTimer);
-            prevTorrentFilesFilterValue = value;
-            torrentFilesTable.setFilter(value);
-            clearTimeout(torrentFilesFilterInputTimer);
-            torrentFilesFilterInputTimer = setTimeout(function() {
-                if (current_hash === "")
-                    return;
-                torrentFilesTable.updateTable(false);
 
-                if (value.trim() === "")
+        const value = $('torrentFilesFilterInput').get("value");
-                    collapseAllNodes();
+        torrentFilesTable.setFilter(value);
-                else
+
-                    expandAllNodes();
+        torrentFilesFilterInputTimer = setTimeout(() => {
-            }, 400);
+            torrentFilesFilterInputTimer = -1;
-        }
+
+            if (current_hash === "")
+                return;
+
+            torrentFilesTable.updateTable();
+
+            if (value.trim() === "")
+                collapseAllNodes();
+            else
+                expandAllNodes();
+        }, window.qBittorrent.Misc.FILTER_INPUT_DELAY);
     });
 
     /**

src/webui/www/private/views/log.html

@@ -183,7 +183,7 @@
         };
 
         let customSyncLogDataInterval = null;
-        let logFilterTimer;
+        let logFilterTimer = -1;
         let inputtedFilterText = "";
         let selectBox;
         let selectedLogLevels = JSON.parse(LocalPreferences.get('qbt_selected_log_levels')) || ['1', '2', '4', '8'];
@@ -297,9 +297,11 @@
         const logFilterChanged = () => {
             clearTimeout(logFilterTimer);
             logFilterTimer = setTimeout((curTab) => {
+                logFilterTimer = -1;
+
                 tableInfo[curTab].instance.updateTable(false);
                 updateLabelCount(curTab);
-            }, 400, currentSelectedTab);
+            }, window.qBittorrent.Misc.FILTER_INPUT_DELAY, currentSelectedTab);
         };
 
         const setCurrentTab = (tab) => {
@@ -321,6 +323,7 @@
             }
 
             clearTimeout(logFilterTimer);
+            logFilterTimer = -1;
             load();
 
             if (tableInfo[currentSelectedTab].instance.filterText !== getFilterText()) {
@@ -377,6 +380,8 @@
 
                     if (response.length > 0) {
                         clearTimeout(logFilterTimer);
+                        logFilterTimer = -1;
+
                         for (let i = 0; i < response.length; ++i) {
                             let row;
                             if (curTab === 'main') {

src/webui/www/private/views/search.html

@@ -233,7 +233,6 @@
             max: 0.00,
             maxUnit: 3
         };
-        let prevNameFilterValue;
         let selectedCategory = "QBT_TR(All categories)QBT_TR[CONTEXT=SearchEngineWidget]";
         let selectedPlugin = "all";
         let prevSelectedPlugin;
@@ -258,18 +257,17 @@
             searchResultsTable.setup('searchResultsTableDiv', 'searchResultsTableFixedHeaderDiv', searchResultsTableContextMenu);
             getPlugins();
 
-            let searchInNameFilterTimer = null;
             // listen for changes to searchInNameFilter
-            $('searchInNameFilter').addEvent('input', function() {
+            let searchInNameFilterTimer = -1;
-                const value = $('searchInNameFilter').get("value");
+            $('searchInNameFilter').addEvent('input', () => {
-                if (value !== prevNameFilterValue) {
+                clearTimeout(searchInNameFilterTimer);
-                    prevNameFilterValue = value;
+                searchInNameFilterTimer = setTimeout(() => {
-                    clearTimeout(searchInNameFilterTimer);
+                    searchInNameFilterTimer = -1;
-                    searchInNameFilterTimer = setTimeout(function() {
+
-                        searchText.filterPattern = value;
+                    const value = $('searchInNameFilter').get("value");
-                        searchFilterChanged();
+                    searchText.filterPattern = value;
-                    }, 400);
+                    searchFilterChanged();
-                }
+                }, window.qBittorrent.Misc.FILTER_INPUT_DELAY);
             });
 
             new Keyboard({