Add window based on current pos

src/controller/actions/shortcuts.ts

@@ -193,9 +193,11 @@ export class ShortcutManager {
             return;
         }
         if (this.ctrl.windowExtensions.get(window)!.isTiled) {
+            this.ctrl.windowExtensions.get(window)!.shouldTile = false;
             this.ctrl.driverManager.untileWindow(window);
         } else {
-            this.ctrl.driverManager.addWindow(window);
+            this.ctrl.windowExtensions.get(window)!.shouldTile = true;
+            this.ctrl.driverManager.addWindowToPosition(window, null);
         }
         this.ctrl.driverManager.rebuildLayout();
     }

src/controller/actions/windowhooks.ts

@@ -2,7 +2,7 @@
 
 import { MaximizeMode, Tile, Window } from "kwin-api";
 import { Controller } from "../";
-import { GRect } from "../../util/geometry";
+import { GPoint, GRect } from "../../util/geometry";
 import { Log } from "../../util/log";
 import { WindowExtensions } from "../extensions";
 
@@ -24,6 +24,9 @@ export class WindowHooks {
         window.interactiveMoveResizeStepped.connect(
             this.interactiveMoveResizeStepped.bind(this),
         );
+        window.interactiveMoveResizeFinished.connect(
+            this.interactiveMoveResizeFinished.bind(this),
+        );
         window.tileChanged.connect(this.tileChanged.bind(this));
         window.fullScreenChanged.connect(this.fullscreenChanged.bind(this));
         window.minimizedChanged.connect(this.minimizedChanged.bind(this));
@@ -158,32 +161,25 @@ export class WindowHooks {
             this.ctrl.driverManager.rebuildLayout(this.window.output);
         }
     }
+
+    interactiveMoveResizeFinished() {
+        if (
+            this.ctrl.driverManager.buildingLayout ||
+            this.ctrl.driverManager.resizingLayout ||
+            !this.extensions.shouldTile
+        ) {
+            return;
+        }
+        this.putWindowInBestTile(false)
+    }
 
-    putWindowInBestTile(): void {
-        if (this.extensions.lastTiledLocation != null) {
-            // fancy and illegally long code to place tile in a similar position from when it was untiled
-            let tile = this.ctrl.workspace
-                .tilingForScreen(this.window.output)
-                .bestTileForPosition(
-                    this.extensions.lastTiledLocation.x,
-                    this.extensions.lastTiledLocation.y,
-                );
-            // if its null then its root tile (usually)
-            if (tile == null) {
-                tile = this.ctrl.workspace.tilingForScreen(
-                    this.window.output,
-                ).rootTile;
-            }
-            this.ctrl.driverManager.putWindowInTile(
-                this.window,
-                tile,
-                new GRect(tile.absoluteGeometry).directionFromPoint(
-                    this.extensions.lastTiledLocation,
-                ),
-            );
-        } else {
-            this.ctrl.driverManager.addWindow(this.window);
+    putWindowInBestTile(prefer_last: boolean = true): void {
+        let positionToMatch: GPoint|null = null;  // Null uses the middle of the window.
+        if (this.extensions.lastTiledLocation != null && prefer_last) {
+            positionToMatch = this.extensions.lastTiledLocation;
         }
+
+        this.ctrl.driverManager.addWindowToPosition(this.window, positionToMatch);
         this.ctrl.driverManager.rebuildLayout(this.window.output);
     }
 
@@ -208,7 +204,7 @@ export class WindowHooks {
             !this.window.minimized &&
             !(this.extensions.maximized && this.extensions.isSingleMaximized)
         ) {
-            this.putWindowInBestTile();
+            this.putWindowInBestTile(true);
         }
     }
 

src/controller/extensions.ts

@@ -56,6 +56,7 @@ export class WindowExtensions {
     private previousDesktopsInternal: Desktop[] = [];
     isTiled: boolean = false; // not is in a tile, but is registered in engine
     wasTiled: boolean = false; // windows that were tiled when they could be (minimized/maximized/fullscreen)
+    shouldTile: boolean = true;
     lastTiledLocation: GPoint | null = null;
     clientHooks: WindowHooks | null = null;
     isSingleMaximized: boolean = false; // whether the window is solo maximized or not (in accordance with maximize single windows)

src/driver/index.ts

@@ -3,8 +3,8 @@
 import { TilingDriver } from "./driver";
 import { EngineConfig, TilingEngineFactory } from "../engine";
 import { Window, Tile, Output } from "kwin-api";
-import { QTimer } from "kwin-api/qt";
-import { Direction } from "../util/geometry";
+import { QPoint, QTimer } from "kwin-api/qt";
+import { Direction, GRect } from "../util/geometry";
 import { Controller } from "../controller";
 import { Log } from "../util/log";
 import { Config, Borders } from "../util/config";
@@ -262,6 +262,34 @@ export class DriverManager {
         }
     }
 
+    addWindowToPosition(window: Window, pos: QPoint|null, desktops?: Desktop[]): void {
+        // If no position is given, use the middle of the window.
+        if (pos == null) {
+            pos = {
+                x: window.pos.x + window.size.width / 2,
+                y: window.pos.y + window.size.height / 2
+            };
+        }
+
+        let tile = this.ctrl.workspace
+            .tilingForScreen(window.output)    
+            .bestTileForPosition(pos.x, pos.y);
+
+        // if its null then its root tile (usually)
+        if (tile == null) {
+            tile = this.ctrl.workspace.tilingForScreen(
+                window.output,
+            ).rootTile;
+        }
+        this.putWindowInTile(
+            window,
+            tile,
+            new GRect(tile.absoluteGeometry).directionFromPoint(
+                pos,
+            ),
+        );
+    }
+
     addWindow(window: Window, desktops?: Desktop[]): void {
         if (desktops == undefined) {
             desktops =