kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-08-22 01:40:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

[dolphin/search] Avoid unnecesary reloads on setSearchPath

Browse source 
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
This commit is contained in:
Ismael Asensio 2019-10-07 21:50:59 +02:00 committed by Elvis Angelaccio
parent 304ab14e08
commit e30357343b
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/search/dolphinsearchbox.cpp
View file

@ -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());