https://bugs.kde.org/show_bug.cgi?id=390295
Vlad Zagorodniy <vladz...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://commits.kde.org/kwi
| |n/a9e469bf13f6d99c24eb2ae24
| |c3a0f4ec0a79d61
Version Fixed In| |5.15.3
Status|CONFIRMED |RESOLVED
Resolution|--- |FIXED
--- Comment #3 from Vlad Zagorodniy <vladz...@gmail.com> ---
Git commit a9e469bf13f6d99c24eb2ae24c3a0f4ec0a79d61 by Vlad Zagorodniy.
Committed on 08/03/2019 at 08:38.
Pushed by vladz into branch 'Plasma/5.15'.
Properly restore current desktop from session
Summary:
VirtualDesktopManager is initialized in two places: Workspace::init and
Workspace::initWithX11. The former method loads virtual desktops from
the config file and the latter method synchronizes VirtualDesktopManager
with RootInfo.
Both methods do
if (!VirtualDesktopManager::self()->setCurrent(m_initialDesktop))
VirtualDesktopManager::self()->setCurrent(1);
which makes sense in Workspace::init, but not in Workspace::initWithX11.
When Workspace::initWithX11 is called, the current virtual desktop is
the same as m_initialDesktop. So that piece of code basically makes
the first virtual desktop current no matter what.
FIXED-IN: 5.15.3
Reviewers: #kwin, davidedmundson
Reviewed By: #kwin, davidedmundson
Subscribers: kwin
Tags: #kwin
Differential Revision: https://phabricator.kde.org/D19520
M +3 -3 workspace.cpp
https://commits.kde.org/kwin/a9e469bf13f6d99c24eb2ae24c3a0f4ec0a79d61
--
You are receiving this mail because:
You are watching all bug changes.
