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

Reply via email to