Store normal clients and desktop clients in the same list
ClosedPublic

Authored by zzag on May 8 2020, 8:12 AM.

Details

Summary

On X11, Workspace stores windows in two lists. One with desktop windows
and the other one with all other windows. On Wayland, desktop windows
and normal windows are stored in the same list - m_allClients.

In order to unify scripting on X11 and Wayland, this change makes the
Workspace class store X11 desktop windows and normal X11 windows in the
same list. It's the responsibility of scripts to filter desktop windows.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.May 8 2020, 8:12 AM
Restricted Application added a project: KWin. · View Herald TranscriptMay 8 2020, 8:12 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.May 8 2020, 8:12 AM
zzag edited the summary of this revision. (Show Details)May 8 2020, 8:57 AM
apol accepted this revision.May 10 2020, 3:42 PM
apol added a subscriber: apol.

Looks good to me. Maybe to merge after branching?

This revision is now accepted and ready to land.May 10 2020, 3:42 PM
zzag added a comment.May 10 2020, 8:14 PM
In D29522#667592, @apol wrote:

Looks good to me. Maybe to merge after branching?

That was my plan. :-)

This revision was automatically updated to reflect the committed changes.