┌   ┐
54
└   ┘

summaryrefslogtreecommitdiff
path: root/src/search
diff options
context:
space:
mode:
authorIsmael Asensio <[email protected]>2019-10-07 21:50:59 +0200
committerElvis Angelaccio <[email protected]>2019-10-07 21:50:59 +0200
commite30357343bdadd6bf6bb94e5adf54d8b840ef1bd (patch)
tree6f663966e66c859a8f8c234e136a57f28bd6ef4f /src/search
parent304ab14e0883b0c601dc3c7f7c26387245db66e0 (diff)
[dolphin/search] Avoid unnecesary reloads on setSearchPath
Summary: Check in `setSearchPath()` if the `url` passed is already the `searchPath` This was causing an effect where reloading the same search URL results on buttons `From Here/Everywhere` toggling. See D24422 Test Plan: On your home folder press `Ctrl-F` several times Buttons `From Here/Everywhere` should now not toggle each time Reviewers: #dolphin, elvisangelaccio, meven, ngraham Reviewed By: #dolphin, ngraham Subscribers: kfm-devel Tags: #dolphin Differential Revision: https://phabricator.kde.org/D24449
Diffstat (limited to 'src/search')
-rw-r--r--src/search/dolphinsearchbox.cpp4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/search/dolphinsearchbox.cpp b/src/search/dolphinsearchbox.cpp
index f3dd20462..6a87edc9e 100644
--- a/src/search/dolphinsearchbox.cpp
+++ b/src/search/dolphinsearchbox.cpp
@@ -81,6 +81,10 @@ QString DolphinSearchBox::text() const
void DolphinSearchBox::setSearchPath(const QUrl& url)
{
+ if (url == m_searchPath) {
+ return;
+ }
+
m_searchPath = url;
QFontMetrics metrics(m_fromHereButton->font());