Add-app-to-panel string overhaul.
ClosedPublic

Authored by hein on Jan 4 2017, 8:32 AM.

Details

Summary
  • Get rid of the unwieldy "Show a launcher when not running" and use Pin instead
  • Clarify that "Add to Panel" adds a widget to distinguish from adding to Task Manager
  • Use "Pin to Task Manager" in launchers as well

Often-discussed, often-desired, VDG-approved.

BUG:368549

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
hein updated this revision to Diff 9690.Jan 4 2017, 8:32 AM
hein retitled this revision from to Add-app-to-panel string overhaul..
hein updated this object.
hein edited the test plan for this revision. (Show Details)
hein added reviewers: Plasma, mart, broulik.
hein added a subscriber: plasma-devel.
Restricted Application added a project: Plasma. · View Herald TranscriptJan 4 2017, 8:32 AM
mart accepted this revision.Jan 4 2017, 12:31 PM
mart edited edge metadata.
This revision is now accepted and ready to land.Jan 4 2017, 12:31 PM
broulik added inline comments.Jan 4 2017, 12:40 PM
applets/kicker/plugin/actionlist.cpp
144

Perhaps "(Icon Widget)", or "Add to Panel as Widget"?

applets/taskmanager/package/contents/ui/ContextMenu.qml
409

Please add some i18nc here, I fear it might become "Stecknadel" or something like this otherwise

This revision was automatically updated to reflect the committed changes.
hein added inline comments.Jan 5 2017, 5:22 AM
applets/kicker/plugin/actionlist.cpp
144

I decided against naming the widget since we kinda hide the existence of Icon ..

applets/taskmanager/package/contents/ui/ContextMenu.qml
409

Will do.