[plasma-desktop] /: Call KRunner KRunner
Nate Graham
null at kde.org
Wed May 22 19:19:25 BST 2019
Git commit d0c8fe93982e86b89bcd416830fab33d091001a7 by Nate Graham.
Committed on 22/05/2019 at 18:19.
Pushed by ngraham into branch 'master'.
Call KRunner KRunner
Summary:
Plasma uses inconsistent terminology to refer to KRunner, calling it "Run Command" in
Kicker/Kickoff/etc, but "Plasma Search" in System Settings. For reasons explained in
{T10966} neither term is ideal, and the inconsistency is bad too.
This patch unifies on "Show KRunner" for the action and "KRunner" for the KCM name,
transforming it into a recognizable brand name like Apple's Spotlight.
This patch also makes the icons consistent and adjusts some strings to more clearly
indicate what KRunner is and does.
Test Plan:
{F6842587}
{F6842586}
Reviewers: #vdg, #plasma, ndavis
Reviewed By: #vdg, ndavis
Subscribers: ndavis, plasma-devel, kde-doc-english
Tags: #plasma, #documentation
Maniphest Tasks: T10966
Differential Revision: https://phabricator.kde.org/D21343
M +3 -3 applets/kicker/plugin/computermodel.cpp
M +1 -1 design/containmentactions
M +1 -1 doc/plasma-desktop/index.docbook
M +1 -1 kcms/runners/kcm.cpp
M +1 -1 kcms/runners/kcm_plasmasearch.desktop
https://commits.kde.org/plasma-desktop/d0c8fe93982e86b89bcd416830fab33d091001a7
diff --git a/applets/kicker/plugin/computermodel.cpp b/applets/kicker/plugin/computermodel.cpp
index 4e6272c2..7227174d 100644
--- a/applets/kicker/plugin/computermodel.cpp
+++ b/applets/kicker/plugin/computermodel.cpp
@@ -101,11 +101,11 @@ QVariant RunCommandModel::data(const QModelIndex &index, int role) const
}
if (role == Qt::DisplayRole) {
- return i18n("Run Command...");
+ return i18n("Show KRunner");
} else if (role == Qt::DecorationRole) {
- return QIcon::fromTheme(QStringLiteral("system-run"));
+ return QIcon::fromTheme(QStringLiteral("plasma-search"));
} else if (role == Kicker::DescriptionRole) {
- return i18n("Run a command or a search query");
+ return i18n("Search, calculate, or run a command");
} else if (role == Kicker::GroupRole) {
return i18n("Applications");
}
diff --git a/design/containmentactions b/design/containmentactions
index c1c201d1..492e2dad 100644
--- a/design/containmentactions
+++ b/design/containmentactions
@@ -63,7 +63,7 @@ Containment Interface
The Containment class supports loading and using containmentactions plugins.
Subclasses need do nothing to get this support. If a subclass has extra actions it wants in the
contextmenu, it can provide them in contextualActions() the same as before - but there's no longer
-any need to return standard actions like "add widgets" and "run command".
+any need to return standard actions like "add widgets" and "show krunner".
ContextActions plugins to use are set using the setContainmentActions(const QString &trigger, const QString &pluginName)
method. an empty string removes any plugin set for the given trigger.
diff --git a/doc/plasma-desktop/index.docbook b/doc/plasma-desktop/index.docbook
index 9b10bde8..64691188 100644
--- a/doc/plasma-desktop/index.docbook
+++ b/doc/plasma-desktop/index.docbook
@@ -548,7 +548,7 @@ navigate through the matches using the tab and arrow keys.
</screenshot>
<para>Use the <inlinemediaobject><imageobject><imagedata fileref="krunner-configure.png" format="PNG"/>
-</imageobject></inlinemediaobject> button to open the <guilabel>Plasma Search</guilabel> &systemsettings;
+</imageobject></inlinemediaobject> button to open the <guilabel>KRunner</guilabel> &systemsettings;
module and configure where and what to search for in &krunner;
</para>
diff --git a/kcms/runners/kcm.cpp b/kcms/runners/kcm.cpp
index 555cd65c..3ce5f4f3 100644
--- a/kcms/runners/kcm.cpp
+++ b/kcms/runners/kcm.cpp
@@ -56,7 +56,7 @@ SearchConfigModule::SearchConfigModule(QWidget* parent, const QVariantList& args
QHBoxLayout *headerLayout = new QHBoxLayout(this);
- QLabel *label = new QLabel(i18n("Select the search plugins:"));
+ QLabel *label = new QLabel(i18n("Enable or disable KRunner plugins:"));
QPushButton *clearHistoryButton = new QPushButton(i18n("Clear History"));
clearHistoryButton->setIcon(QIcon::fromTheme(isRightToLeft() ? QStringLiteral("edit-clear-locationbar-ltr")
diff --git a/kcms/runners/kcm_plasmasearch.desktop b/kcms/runners/kcm_plasmasearch.desktop
index 96b7000c..edb4466b 100644
--- a/kcms/runners/kcm_plasmasearch.desktop
+++ b/kcms/runners/kcm_plasmasearch.desktop
@@ -10,7 +10,7 @@ X-KDE-ParentApp=kcontrol
X-KDE-System-Settings-Parent-Category=search
X-KDE-Weight=20
-Name=Plasma Search
+Name=KRunner
Name[ar]=بحث بلازما
Name[bs]=Plazma pretraga
Name[ca]=Cerca de Plasma
More information about the kde-doc-english
mailing list