Compiz

Merge lp:~muktupavels/compiz/fix-initial-server-frame-position into lp:compiz

  1. fix-initial-server-frame-position
  2. Merge into 0.9.13
Proposed by Alberts Muktupāvels on 2017-04-20
Status: Merged
Approved by: Marco Trevisan (Treviño) on 2017-04-20
Approved revision: 4108
Merged at revision: 4111
Proposed branch: lp:~muktupavels/compiz/fix-initial-server-frame-position
Merge into: lp:compiz
Prerequisite: lp:~muktupavels/compiz/require-libmetacity-3-22
Diff against target: 14 lines (+2/-2)
1 file modified
src/window.cpp (+2/-2)
To merge this branch: bzr merge lp:~muktupavels/compiz/fix-initial-server-frame-position
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) 2017-04-20 Approve on 2017-04-20
Review via email: mp+322817@code.launchpad.net

Commit Message

Fix initial serverFrame position.

Description of the Change

Fix initial serverFrame position.

I was not able to find bug for this on launchpad, but there is one for MATE:
https://github.com/mate-desktop/mate-panel/issues/249

This happens not only with mate-panel or gnome-panel, but also CSD windows. serverFrame is put at 0x0 position and must be clicked to be moved to correct position.

To post a comment you must log in.
Marco Trevisan (Treviño) (3v1n0) wrote :

Seems fair.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/window.cpp'
2--- src/window.cpp 2016-05-17 02:52:25 +0000
3+++ src/window.cpp 2017-04-20 07:53:31 +0000
4@@ -6921,8 +6921,8 @@
5 /* Gravity here is assumed to be SouthEast, clients can update
6 * that if need be */
7
8- serverFrameGeometry.set (serverInput.left - border.left,
9- serverInput.top - border.top,
10+ serverFrameGeometry.set (wa.x + (serverInput.left - border.left),
11+ wa.y + (serverInput.top - border.top),
12 wa.width + (serverInput.left +
13 serverInput.right),
14 wa.height + (serverInput.top +

Subscribers

People subscribed via source and target branches