https://bugs.kde.org/show_bug.cgi?id=437466

Nicolas Fella <nicolas.fe...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kwin/commit/7f36f0124787 |ma/kwin/commit/c9329ec6d99d
                   |22e0cda859b28fdd7cd2d63f422 |5f198fbb88d5dfd714c1d38a962
                   |5                           |4

--- Comment #6 from Nicolas Fella <nicolas.fe...@gmx.de> ---
Git commit c9329ec6d99d5f198fbb88d5dfd714c1d38a9624 by Nicolas Fella.
Committed on 24/05/2021 at 11:54.
Pushed by nicolasfella into branch 'Plasma/5.22'.

[kcms/virtualdesktops] Properly track synchronizing state

When the user applies a change like adding a desktop
DesktopsModel::syncWithServer is called and m_synchronizing is set to
true. The server then sends some kind of response (e.g. desktopCreated)
and updateModifiedState is called which sets m_synchronizing to false
again.

However if a setting is changed that does not trigger any server change
(e.g. the show OSD setting) m_synchonizing is never set to false again,
causing problems down the line.

Instead of relying on updateModifiedState to reset m_synchronizing track
the syncronizing state by refcounting the DBus calls
(cherry picked from commit 7f36f012478722e0cda859b28fdd7cd2d63f4225)

M  +8    -10   src/kcmkwin/kwindesktop/desktopsmodel.cpp
M  +1    -1    src/kcmkwin/kwindesktop/desktopsmodel.h

https://invent.kde.org/plasma/kwin/commit/c9329ec6d99d5f198fbb88d5dfd714c1d38a9624

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to