[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kate] addons/search: search plugin: fix startSearch() when pressing Enter in the folder requester
From:       Alex Neundorf <null () kde ! org>
Date:       2017-09-01 20:45:03
Message-ID: E1dnsoJ-0008Jz-Fa () code ! kde ! org
[Download RAW message or body]

Git commit ab94e071a7bfbc03bbe9032c530e7b77b784e865 by Alex Neundorf.
Committed on 01/09/2017 at 20:42.
Pushed by neundorf into branch 'master'.

search plugin: fix startSearch() when pressing Enter in the folder requester

startSearch() was connected to KUrlComboRequester::returnPressed(), but som=
ehow this
signal is not emitted. The signals are not blocked, and the eventFilter als=
o doesn't
seem to interfer.
Connecting to the signal of the lineedit makes it work...

Alex

M  +2    -1    addons/search/plugin_search.cpp

https://commits.kde.org/kate/ab94e071a7bfbc03bbe9032c530e7b77b784e865

diff --git a/addons/search/plugin_search.cpp b/addons/search/plugin_search.=
cpp
index 8b45d3c2e..707baed61 100644
--- a/addons/search/plugin_search.cpp
+++ b/addons/search/plugin_search.cpp
@@ -317,7 +317,8 @@ m_mainWindow (mainWin)
     connect(&m_changeTimer, SIGNAL(timeout()), this, SLOT(startSearchWhile=
Typing()));
 =

     connect(m_ui.searchCombo->lineEdit(), SIGNAL(returnPressed()), this, S=
LOT(startSearch()));
-    connect(m_ui.folderRequester,  SIGNAL(returnPressed()), this, SLOT(sta=
rtSearch()));
+// connecting to returnPressed() of the folderRequester doesn't work, I ha=
ven't found out why yet. But connecting to the linedit works:
+    connect(m_ui.folderRequester->comboBox()->lineEdit(),  SIGNAL(returnPr=
essed()), this, SLOT(startSearch()));
     connect(m_ui.filterCombo,      SIGNAL(returnPressed()), this, SLOT(sta=
rtSearch()));
     connect(m_ui.excludeCombo,     SIGNAL(returnPressed()), this, SLOT(sta=
rtSearch()));
     connect(m_ui.searchButton,     SIGNAL(clicked()),       this, SLOT(sta=
rtSearch()));
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic