ngraham added inline comments.

INLINE COMMENTS

> batterymonitor.qml:142
> +        if (batterymonitor.kcmEnergyInformationAuthorized) {
> +            plasmoid.setAction("energyinformationkcm", i18n("&Energy 
> Information..."), "battery");
> +        }

Generally we start menu items with action verbs. And in this case we don't need 
ellipsis because no further input from the user is required as a part of the 
action. See 
https://hig.kde.org/style/writing/labels.html#using-ellipses-in-labels

So for this string, I would recommend "Show Energy Information"

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D20533

To: meven, #plasma, ngraham, broulik, #vdg
Cc: abetts, apol, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, sebas, mart

Reply via email to