diff options
| author | srdusr <[email protected]> | 2024-08-09 10:34:00 +0200 |
|---|---|---|
| committer | srdusr <[email protected]> | 2024-08-09 10:34:00 +0200 |
| commit | 09e63d204132f767cee7d0846e291d659f79fe90 (patch) | |
| tree | db861dc95cbe9db9e628beb351c8b9d977046a09 /legacy-cpp/scripts | |
| parent | 31de54096243fb000fa6e7008a914861293482a5 (diff) | |
| download | srdwm-09e63d204132f767cee7d0846e291d659f79fe90.tar.gz srdwm-09e63d204132f767cee7d0846e291d659f79fe90.zip | |
Fix closing an XWayland window doing nothing on both backends
Platform::close only ever called w.toplevel(), which is None for an
XWayland window - closing one (the WM's own close binding, or `srd
dispatch close`) silently did nothing at all, on both udev and winit.
Found live: a leftover untitled fullscreen window wouldn't close via
srd dispatch close even after several seconds, tracing back to this.
Fixed by falling back to X11Surface::close() when there's no xdg
toplevel - it already handles both cases smithay-side (a polite
WM_DELETE_WINDOW for a cooperating client, outright destroy_window for
one that doesn't support it), so no new logic was needed, just calling
it.
Diffstat (limited to 'legacy-cpp/scripts')
0 files changed, 0 insertions, 0 deletions