tris/recoll
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Avoid having to type CR twice to search in some cases

Browse Source 
Happened for multiword searches on the second and further term because the
popup ate the CR but did not fire as there was no active match.
This commit is contained in:
Jean-Francois Dockes 2020-05-06 19:06:14 +02:00
parent ef1b8343fa
commit d8edbcbc55
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/qtgui/ssearch_w.cpp
View File

@ -250,7 +250,8 @@ void SSearch::onCompleterShown()
// This is to avoid that if the user types Backspace or Del while we
// have inserted / selected the current completion, the lineedit text
// goes back to what it was, the completion fires, and it looks like
// nothing was typed. So we disable the completion if a
// nothing was typed. Disable the completionn after Del or Backspace
// is typed.
bool SSearch::eventFilter(QObject *target, QEvent *event)
{
Q_UNUSED(target);
@ -419,7 +420,9 @@ void SSearch::searchTypeChanged(int typ)
void SSearch::startSimpleSearch()
{
if (queryText->completer() && queryText->completer()->popup()->isVisible()) {
// Avoid a double search if we are fired on CR and the completer is active
if (queryText->completer() && queryText->completer()->popup()->isVisible()
&& !queryText->completer()->currentCompletion().isEmpty()) {
return;
}
string u8 = qs2u8s(queryText->text());