Differential D17096

[scripting] Port the remaining bits of API to AbstractClient
AbandonedPublic
Actions

Authored by zzag on Nov 22 2018, 10:39 AM.

Details

Reviewers
None
Group Reviewers
KWin
Summary

Currently, clientArea(ClientAreaOption, const Client *) and clientList()
are quite X11-specific, which results in having some tiling scripts not
working with Wayland clients. This change is the first step towards working
tiling scripts in the Wayland session.

Diff Detail

Repository
R108 KWin
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 10243
Build 10261: arc lint + arc unit
zzag created this revision.Nov 22 2018, 10:39 AM
Restricted Application added a project: KWin. · View Herald TranscriptNov 22 2018, 10:39 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Nov 22 2018, 10:39 AM
zzag abandoned this revision.Nov 23 2018, 11:37 AM
zzag reclaimed this revision.Nov 26 2018, 9:02 AM
davidedmundson added a subscriber: davidedmundson.Feb 20 2019, 12:56 PM

Consider this as "accepted", pending D17097 getting accepted as it has that same discussion.

scripting/workspace_wrapper.cpp
92–93

this #define is now a bit overkill for one method!

zzag updated this revision to Diff 55022.Mar 29 2019, 10:22 AM

Drop GETTER macro

zzag marked an inline comment as done.Mar 29 2019, 10:22 AM
zzag abandoned this revision.May 23 2019, 9:41 PM

Hmm, looks like arc diff forgot to update commit message. In either case, this change is in master.

Revision Contents
Changeset List

Diff 55022

View Options

scripting/meta.cpp

Loading...
View Options

scripting/workspace_wrapper.h

Loading...
View Options

scripting/workspace_wrapper.cpp

Loading...