Differential D29520

Merge setting up abstractclient and x11client connections in EffectsHandler
ClosedPublic
Actions

Authored by zzag on May 8 2020, 7:24 AM.

Details

Reviewers
apol
Group Reviewers
KWin
Commits
R108:78180c03484f: Merge setting up abstractclient and x11client connections in EffectsHandler
Summary

Toplevel::paddingChanged() is not an X11-specific signal.

Test Plan

Relevant tests still pass.

Diff Detail

Repository
R108 KWin
Branch
effects-setup-client-connections
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26536
Build 26554: arc lint + arc unit
zzag created this revision.May 8 2020, 7:24 AM
Restricted Application added a project: KWin. · View Herald TranscriptMay 8 2020, 7:24 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.May 8 2020, 7:25 AM
apol accepted this revision.May 10 2020, 5:36 PM
apol added a subscriber: apol.

Good catch

This revision is now accepted and ready to land.May 10 2020, 5:36 PM
Closed by commit R108:78180c03484f: Merge setting up abstractclient and x11client connections in EffectsHandler (authored by zzag). · Explain WhyMay 11 2020, 10:28 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Meffects.h (4 lines)
Meffects.cpp (17 lines)

Diff 82241

View Options

effects.h

Loading...
View Options

effects.cpp

Loading...