Use "Do x" action texts for drop menu entries
ClosedPublic

Authored by kossebau on Apr 16 2018, 1:29 PM.

Details

Summary

Dropping some item onto a containment results in a menu which lists widgets
and wallpaper plugins without explicitly telling what will happen onto
activation.

Changing the menu entries into "Add widget" and "Set wallpaper" makes it
more obvious what they will result in.

Diff Detail

Repository
R242 Plasma Framework (Library)
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Apr 16 2018, 1:29 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 16 2018, 1:29 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
kossebau requested review of this revision.Apr 16 2018, 1:29 PM

Example of dropping an SVG file onto the background:

Before:


After:

Alternativly one could change those menu section titles (which sadly are hard to recognize as such with Breeze theme) to be "Add Widgets" and "Set Wallpaper", and then list the widgets and wallpapers by their name as before, as kind of inlined submenus.
Though that approach might conflict with some other potential entries in that menu, with ""Plasma Package" for the section and "Install" as menu entry. Possibly might need a complete overhaul then.

Alternativly one could change those menu section titles (which sadly are hard to recognize as such with Breeze theme)

Agreed, this definitely needs improvement.

to be "Add Widgets" and "Set Wallpaper", and then list the widgets and wallpapers by their name as before, as kind of inlined submenus.
Though that approach might conflict with some other potential entries in that menu, with ""Plasma Package" for the section and "Install" as menu entry. Possibly might need a complete overhaul then.

I prefer your original approach of simply adding verbs.

davidedmundson accepted this revision.Apr 25 2018, 1:23 PM
This revision is now accepted and ready to land.Apr 25 2018, 1:23 PM
This revision was automatically updated to reflect the committed changes.