From 167d5184eda19f8ce549d6a645c800ab39bce936 Mon Sep 17 00:00:00 2001 From: Ismael Asensio Date: Sun, 20 Oct 2019 11:43:32 +0200 Subject: [dolphin/search] Fix search behavior when selecting "Your Files" Summary: Fix the search box forgetting the location where the user was previously searching into. To do a search on "All Files" instead of setting the `m_searchPath` to `$HOME`, it checks the button state, so the "From Here" location is not lost. As an added benefit, selecting "Your Files" when in a non-indexed folder will use the baloo search instead of a fully non-indexed search from $HOME. This issue is the last remaining one of the series started with D24422, with the purpose of fixing the searchbox parsing and update. Test Plan: - Toggle between "From Here/Your Files" and navigate between locations - The search box remembers the location and keeps a coherent state {F7575402} Reviewers: #dolphin, elvisangelaccio Subscribers: kfm-devel Tags: #dolphin Differential Revision: https://phabricator.kde.org/D24577 --- src/search/dolphinsearchbox.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/search/dolphinsearchbox.h') diff --git a/src/search/dolphinsearchbox.h b/src/search/dolphinsearchbox.h index e70317450..35b422631 100644 --- a/src/search/dolphinsearchbox.h +++ b/src/search/dolphinsearchbox.h @@ -62,8 +62,8 @@ public: QString text() const; /** - * Sets the current path that is used as root for - * searching files, if "From Here" has been selected. + * Sets the current path that is used as root for searching files. + * If @url is the Home dir, "From Here" is selected instead. */ void setSearchPath(const QUrl& url); QUrl searchPath() const; -- cgit v1.3.1