D6469: Terminology fixes in KCMs: KDE-as-the-desktop -> Plasma

[Lukáš Tinkl]

lukas added inline comments.

INLINE COMMENTS

> ltoscano wrote in autostart.cpp:75
> Should this be "Session" or "Plasma Session"? Aren't the settings changed here valid for all desktops which follow the XDG spec?

Yes, it should be valid for all DEs (I'd vote for "Session" only then)

> emailclientconfig_ui.ui:12
>       <property name="whatsThis">
> -      <string>Kmail is the standard Mail program for the KDE desktop.</string>
> +      <string>Kmail is the standard Mail program for the Plasma desktop.</string>
>       </property>

I'd keep the old one as well, just for a better discoverability

> ltoscano wrote in globalpaths.cpp:127
> I explicitly wrote "Plasma" because this setting seems to be Plasma-specific. Did I understand it correctly?

Again, this should be desktop agnostic

REPOSITORY
  R119 Plasma Desktop

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

To: ltoscano
Cc: lukas, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170703/113e118a/attachment.html>