]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Change "Empty Search" to just "Search" to handle searches with no search text
authorNathaniel Graham <pointedstick@zoho.com>
Wed, 13 Dec 2017 05:52:21 +0000 (22:52 -0700)
committerNathaniel Graham <pointedstick@zoho.com>
Wed, 13 Dec 2017 16:19:44 +0000 (09:19 -0700)
Summary:
BUG: 387854

The pre-defined searches (Documents, Videos, etc) all default to no search text. https://cgit.kde.org/dolphin.git/commit/?id=702429ac8e64a5aff4e110a56f526b96ecb795fd changed their window title from "baloosearch - /" to "Empty Search", but the search isn't really empty; it just lacks a search string. It's still looking for documents, or videos, or whatever.

Test Plan:
Is it too late to get this into 5.12?

Tested in KDE Neon. Before:
{F5545170}

After:
{F5545171}

Reviewers: #dolphin, elvisangelaccio, broulik

Reviewed By: broulik

Differential Revision: https://phabricator.kde.org/D9305

src/dolphinmainwindow.cpp

index 067d4a6e1d176d038f1dab966d8e4c14ad546d68..7b3d63fd6e999ef5a71f77d9a29b4d86a11ecd6a 100644 (file)
@@ -1003,7 +1003,7 @@ void DolphinMainWindow::setUrlAsCaption(const QUrl& url)
 
     if (m_activeViewContainer->isSearchModeEnabled()) {
         if(m_activeViewContainer->currentSearchText().isEmpty()){
-            setWindowTitle(i18n("Empty Search"));
+            setWindowTitle(i18n("Search"));
         } else {
             const auto searchText = i18n("Search for %1", m_activeViewContainer->currentSearchText());
             setWindowTitle(searchText);