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

Update searchbox on URL changes (Fix D24369)

Browse source 
Summary:
In D24369 I introduced a regression: Where now the searchbox is updated on url changes for search urls, it is not updated anymore for regular folders. The effect is that not-indexed folders can have the extra options enabled or vice-versa. This patch fixes it, moving back the updating logic to `setSearchModeEnable`.

Test Plan: - Searchbox updates properly when changing between search urls and regular folders, either indexed or nor-indexed.

Reviewers: #dolphin, elvisangelaccio

Subscribers: kfm-devel, elvisangelaccio

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D24376
This commit is contained in:
Ismael Asensio 2019-10-03 22:00:24 +02:00 committed by Elvis Angelaccio
parent c71c1c94be
commit 3d65fa9a79
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/dolphinviewcontainer.cpp
View file

@ -385,6 +385,11 @@ bool DolphinViewContainer::isFilterBarVisible() const
void DolphinViewContainer::setSearchModeEnabled(bool enabled)
{
if (enabled) {
const QUrl& locationUrl = m_urlNavigator->locationUrl();
m_searchBox->fromSearchUrl(locationUrl);
}
if (enabled == isSearchModeEnabled()) {
if (enabled && !m_searchBox->hasFocus()) {
m_searchBox->setFocus();
@ -673,15 +678,11 @@ void DolphinViewContainer::slotUrlNavigatorLocationChanged(const QUrl& url)
slotReturnPressed();
if (KProtocolManager::supportsListing(url)) {
const bool searchUrl = isSearchUrl(url);
if (searchUrl) {
m_searchBox->fromSearchUrl(url);
}
setSearchModeEnabled(searchUrl);
setSearchModeEnabled(isSearchUrl(url));
m_view->setUrl(url);
tryRestoreViewState();
if (m_autoGrabFocus && isActive() && !searchUrl) {
if (m_autoGrabFocus && isActive() && !isSearchUrl(url)) {
// When an URL has been entered, the view should get the focus.
// The focus must be requested asynchronously, as changing the URL might create
// a new view widget.