Some KWin scripts may need to access desktop windows Clients, to get their windowId's for example. And workspace.clientList doesn't include desktop windows on X11.
Details
Details
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
Hmm, this won't work on Wayland because desktop windows are mixed together with normal windows in workspace.clients. The only way to get a list of desktop windows is to filter them, e.g.
const desktops = workspace.clients.filter(client => client.desktopWindow);
I don't see any benefit from splitting X11 windows by type, could we just put desktop windows and other windows in a single list?
Comment Actions
I just thought of a separate list to avoid impact in possible existing scripts that currently do not filter desktop windows and may need to.
But I agree that one list is better than two. And I think I can make that change.
Comment Actions
Well, we're in a delicate situation here. If those scripts want to work on Wayland properly, they need to filter out desktop windows.