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
Branch
misplaced-client-window
Lint
Lint SkippedExcuse: foo
Unit
No Unit Test Coverage
Build Status
Buildable 22570
Build 22588: arc lint + arc unit
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 75823

View Options

x11client.cpp

Loading...