From 530b165ddf403269c1eabeaf9940e67f8334e763 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Wed, 20 Sep 2017 00:59:25 +0200 Subject: [PATCH] nativeWindowPlacement: Adjust to overview changes title._spacing is no longer defined, so we end up with bogus positions when window-captions-on-top is set to true. Adjust the positioning to do without that for now, though the whole extension could use a rewrite to not copy everything-and-the-kitching-sink, or be killed off as yet another extension from the original random collection that turned out too expensive to keep dragging along ... https://bugzilla.gnome.org/show_bug.cgi?id=787604 --- extensions/native-window-placement/extension.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/extensions/native-window-placement/extension.js b/extensions/native-window-placement/extension.js index da911bac..998267e9 100644 --- a/extensions/native-window-placement/extension.js +++ b/extensions/native-window-placement/extension.js @@ -431,8 +431,8 @@ function enable() { let titleX = cloneX + (cloneWidth - titleWidth) / 2; /// this is the actual difference to original gnome-shell: - //let titleY = cloneY + cloneHeight + title._spacing; - let titleY = cloneY - title.height + title._spacing; + //let titleY = cloneY + cloneHeight - (title.height - this.borderSize) / 2; + let titleY = cloneY - (title.height - this.borderSize) / 2; if (animate) this._animateOverlayActor(title, Math.floor(titleX), Math.floor(titleY), titleWidth);