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

fixed desktop remove on closing maximized window #13

Closed k-amin07 closed 3 years ago

k-amin07 commented 3 years ago

If you close a maximized window, the virtual desktop does not get removed. Upon debugging I figured that in 'closed' hook on line 181 in main.js, client object is None type. I fixed it by hooking the closed handler with workspace.clientRemoved in install and uninstall function.

k-amin07 commented 3 years ago

Nevermind it started causing kwin crashes. Will look further into it.