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 |https://commits.kde.org/kwi
                   |n/a9e469bf13f6d99c24eb2ae24 |n/f7af113261f7e899e1d954b05
                   |c3a0f4ec0a79d61             |58b2c637a9cced1

--- Comment #5 from Vlad Zagorodniy <vladz...@gmail.com> ---
Git commit f7af113261f7e899e1d954b0558b2c637a9cced1 by Vlad Zagorodniy.
Committed on 13/03/2019 at 09:45.
Pushed by vladz into branch 'Plasma/5.12'.

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/f7af113261f7e899e1d954b0558b2c637a9cced1

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

Reply via email to