D14436: SwitchDesktop mousewheel options with config dialog added

[Thomas Otto]

totto added inline comments.

INLINE COMMENTS

> desktop.cpp:47
> +    : Plasma::ContainmentActions{parent, args}
> +    , m_kwinrcRollOverDesktops{readKwinrcRollOverDesktops()}
>  {

The m_kwinrcRollOverDesktops variable does not work at runtime because two instances of SwitchDesktop are created:

One when plasma-workspace/KDE is started, a second when the settings window is opened. The actual desktop switching is done by the former, but subsequent changes to the kwinrc/RollOverDesktops setting are only seen and set by the latter. So the desktop rollover setting remains what it was when KDE was started.

Is there a way to sync these without writing another config key? Or wait for "Add mechanism to notify other clients of config changes over DBus" / https://phabricator.kde.org/D13034 ?

REPOSITORY
  R120 Plasma Workspace

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

To: totto, hein, broulik, #plasma, davidedmundson
Cc: ngraham, davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181006/fad53b55/attachment.html>