[Differential] [Changed Subscribers] D2345: use a separate configuration per look and feel

graesslin (Martin Gräßlin) noreply at phabricator.kde.org
Thu Aug 11 06:43:51 UTC 2016


graesslin added inline comments.

INLINE COMMENTS

> mart wrote in shellcorona.cpp:550
> I hate this method.
> tough, i need a way from the kcm to reset the current layout to the default.
> it's quite dangerous and exporting a dbus method that effectively destroys the current config is evil...
> other ideas?

You could do it like in KWin: make the KCM trigger a DBus signal which Plasmashell triggers to.

In the end it's going to be the same: one can destroy the current config through dbus, but it's not that exposed. Also I don't think it matters: if a user decides to shoot himself in the foot by manually triggering the DBus signal, let him.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, davidedmundson, #plasma
Cc: graesslin, davidedmundson, ivan, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160811/8720181c/attachment.html>


More information about the Plasma-devel mailing list