diff options
| author | Ismael Asensio <[email protected]> | 2019-10-07 21:50:59 +0200 |
|---|---|---|
| committer | Elvis Angelaccio <[email protected]> | 2019-10-07 21:50:59 +0200 |
| commit | e30357343bdadd6bf6bb94e5adf54d8b840ef1bd (patch) | |
| tree | 6f663966e66c859a8f8c234e136a57f28bd6ef4f | |
| parent | 304ab14e0883b0c601dc3c7f7c26387245db66e0 (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
| -rw-r--r-- | src/search/dolphinsearchbox.cpp | 4 |
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()); |
