The load method and updateLayout ensure the virtual desktop information
is synced to RootInfo and thus to other X11 applications. Thus we need
to call it again when initing X11.
BUG: 385260
fvogt |
KWin | |
Plasma |
The load method and updateLayout ensure the virtual desktop information
is synced to RootInfo and thus to other X11 applications. Thus we need
to call it again when initing X11.
BUG: 385260
Not tested as I'm on Wayland, but given the changes it's obvious.
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
I'll give it a try, currently building. For some reason it fails building for Leap 42.3 though, with undefined references such as:
/home/abuild/rpmbuild/BUILD/kwin-5.10.90git~20171001T152640~252980390/plugins/platforms/virtual/egl_gbm_backend.cpp:174: undefined reference to `KWin::AbstractEglBackend::createContext()'
[ 251s] CMakeFiles/KWinWaylandVirtualBackend.dir/egl_gbm_backend.cpp.o: In function `KWin::EglGbmBackend::init()':
Very likely not related to this diff, probably caused by 42.3 not enabling wayland ("Could NOT find Wayland_Egl").
I gave it a try, it's slightly better but not enough: The virtual desktop pager is not visible by default as there's only a single virtual desktop now instead of two by default.
The single virtual desktop had a name now though and it was possible to edit the count of desktops, but only after the second try for some reason.
So something is still missing.
How does openSUSE set two virtual desktops by default? KWin's default is 1, so this might be completely unrelated...
Interesting, I didn't even know that the default deviates here.
Indeed, the default specified in /etc/xdg/kwinrc is
[Desktops]
Number=2