Directly closing application leaves empty Virtual Desktop

Aetf / kwin-maxmize-to-new-desktop

KWin script that moves fullscreened window to a new virtual desktop

BSD 2-Clause "Simplified" License

55 stars 20 forks source link

Directly closing application leaves empty Virtual Desktop #3

Closed 0kzh closed 5 years ago

0kzh commented 5 years ago

As the title says, directly closing a maximized window that's connected to a virtual desktop currently leaves an empty virtual desktop. It seems that @yuricbraga has already implemented a fix for this in his fork, so it'd be great if we could integrate his changes :)

Aetf commented 5 years ago

That commit was authored by me long time ago and also included here.

The actual problem is KWin crashes and restarts if the clientRemoved handler modifies the number of virtual desktops, leaving the empty virtual desktop not removed.

In the latest commit f05bb5ce794ddca813b956254a8d7380b6c6cde3 I fixed the crash but there are some caveats. KWin obviously have some bugs around its window activation and/or stack ordering, see this.

Anyway, this bug itself is fixed so I'm closing it.