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
Branch
drop-desktop-list
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26538
Build 26556: arc lint + arc unit
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.