Cleanup

contents/code/tile.js

@@ -52,11 +52,6 @@ function Tile(firstClient, tileIndex) {
 		 * resizingStarted and resizingEnded.
 		 */
 		this.resizingStep = new Signal();
-		/**
-		 * Signal which is triggered when the geometry of the tile changes because
-		 * of something different to a user move or resize action.
-		 */
-		this.geometryChanged = new Signal();
 		/**
 		 * Signal which is triggered whenever the tile is moved to a different
 		 * screen. Two parameters are passed to the handlers, the old and the new
@@ -228,8 +223,9 @@ Tile.prototype.onClientGeometryChanged = function(client) {
 				// TEST
 				client.addRepaint(this.rectangle);
 			}
+		} else {
+			print("No rectangle");
 		}
-		this.geometryChanged.emit();
 	} catch(err) {
 		print(err, "in Tile.onClientGeometryChanged");
 	}

contents/code/tilelist.js

@@ -261,6 +261,12 @@ TileList.prototype._onClientTabGroupChanged = function(client) {
 		this._onClientRemoved(client);
 		client.syncTabGroupFor("tiling_floating", true);
 		client.syncTabGroupFor("tiling_tileIndex", true);
+		// What we'd need to do here:
+		// Find the old tile, remove the client from it (and remove it if empty)
+		// Find the new tile, add the client to it
+		// HOWEVER: If the tabgroup has just been created, this is called _twice_ (once for each client)
+		// And the .tabGroup property is always undefined, so there is no way to find the other clients
+		// except for maybe handing it over to tiling and there going over all tiles, removing clients that don't have matching tileIndex
 	} catch(err) {
 		print(err, "in TileList._onClientTabGroupChanged");
 	}

contents/code/tiling.js

@@ -61,7 +61,6 @@ function Tiling(layoutType, desktop, screen) {
  * @return Rectangle which contains the area which shall be used by layouts.
  */
 Tiling.getTilingArea = function(screen, desktop) {
-	print("getTilingArea");
 	return workspace.clientArea(KWin.PlacementArea, screen, desktop);
 };
 
@@ -260,7 +259,6 @@ Tiling.prototype._updateAllTiles = function() {
 	try {
 		// Set the position/size of all tiles
 		if (this.active) {
-			// FIXME: KWin hands us the wrong area if we ask for our real desktop
 			// FIXME: Probable kwin bug: clientArea returns the _former_ area
 			var rect = Tiling.getTilingArea(this.screen, this.desktop);
 			this.layout.setLayoutArea(rect);