From 0ac8ba24c3dec9ce942d4b2a49e17dfe960a4dd2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tor=20Hedin=20Br=C3=B8nner?= Date: Thu, 14 Sep 2017 11:31:56 +0200 Subject: [PATCH] Don't don't ensure window is in view on first-frame Only position the new inactive window correctly, we should never move the focused window without also moving focus, or some explicit move/resize action. --- tiling.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tiling.js b/tiling.js index 6c7f1aa..01cccbf 100644 --- a/tiling.js +++ b/tiling.js @@ -161,7 +161,6 @@ focus_handler = (meta_window, user_data) => { let frame = meta_window.get_frame_rect(); meta_window.move_resize_frame(true, meta_window.scrollwm_initial_position.x, meta_window.scrollwm_initial_position.y, frame.width, frame.height) ensure_viewport(meta_window); - let workspace = workspaces[meta_window.get_workspace().workspace_index]; delete meta_window.scrollwm_initial_position; } else { ensure_viewport(meta_window) @@ -303,8 +302,10 @@ first_frame = (meta_window_actor) => { } let frame = meta_window.get_frame_rect(); meta_window.move_resize_frame(true, meta_window.scrollwm_initial_position.x, meta_window.scrollwm_initial_position.y, frame.width, frame.height) - ensure_viewport(meta_window); + let workspace = workspaces[meta_window.get_workspace().workspace_index]; + propogate_forward(workspace, workspace.indexOf(meta_window) + 1, meta_window.scrollwm_initial_position.x + frame.width); + delete meta_window.scrollwm_initial_position; } }