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

Reply via email to