Differential D27456

Fix misplaced client windows
ClosedPublic
Actions

Authored by zzag on Feb 17 2020, 12:41 PM.

Details

Reviewers
broulik
Group Reviewers
KWin
Commits
R108:63fa01333441: Merge branch 'Plasma/5.18'
R108:cfa5daaad1bf: Fix misplaced client windows
Summary

This regression was introduced by me, sorry. The client window may be at
location other than (0, 0), in which case we have to move it to (0, 0)
to ensure that the client contents is not covered by window frame.

BUG: 417584
FIXED-IN: 5.18.1

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.Feb 17 2020, 12:41 PM
Restricted Application added a project: KWin. · View Herald TranscriptFeb 17 2020, 12:41 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Feb 17 2020, 12:41 PM
broulik added a subscriber: broulik.Feb 17 2020, 12:42 PM

+1 fixes LibreOffice on neon for me

broulik accepted this revision.Feb 18 2020, 8:58 AM
This revision is now accepted and ready to land.Feb 18 2020, 8:58 AM
Closed by commit R108:cfa5daaad1bf: Fix misplaced client windows (authored by zzag). · Explain WhyFeb 18 2020, 9:20 AM
This revision was automatically updated to reflect the committed changes.
davidedmundson added a subscriber: davidedmundson.Feb 18 2020, 10:25 AM

I don't really understand it, but this does match the original code closer and it does make a difference to what we send out over the wire

Belated +1

Revision Contents
Changeset List

PathPackages
Mx11client.cpp (4 lines)

Diff 75895

View Options

x11client.cpp

Loading...