WindowClone: Allocate window icon in allocate virtual

Author: leolost2605

lib/App.vala

@@ -158,8 +158,8 @@ public class Gala.App : GLib.Object {
         return windows.data;
     }
 
-    public GLib.SList<pid_t?> get_pids () {
-        var results = new GLib.SList<pid_t?> ();
+    public GLib.SList<int?> get_pids () {
+        var results = new GLib.SList<int?> ();
         foreach (unowned var window in windows) {
             var pid = window.get_pid ();
             if (pid < 1) {

src/Widgets/WindowClone.vala

@@ -159,8 +159,6 @@ public class Gala.WindowClone : Clutter.Actor {
     }
 
     private void reallocate () {
-        var window_frame_rect = window.get_frame_rect ();
-
         close_button = new Gala.CloseButton (monitor_scale_factor) {
             opacity = 0
         };
@@ -169,7 +167,6 @@ public class Gala.WindowClone : Clutter.Actor {
         window_icon = new WindowIcon (window, WINDOW_ICON_SIZE, (int)Math.round (monitor_scale_factor));
         window_icon.opacity = 0;
         window_icon.set_pivot_point (0.5f, 0.5f);
-        set_window_icon_position (window_frame_rect.width, window_frame_rect.height, monitor_scale_factor);
 
         add_child (close_button);
         add_child (window_icon);
@@ -276,16 +273,9 @@ public class Gala.WindowClone : Clutter.Actor {
 
         unowned var display = window.get_display ();
         var monitor_geom = display.get_monitor_geometry (window.get_monitor ());
-        var initial_scale = monitor_scale_factor;
-        var target_scale = display.get_monitor_scale (window.get_monitor ());
-        var offset_x = monitor_geom.x;
-        var offset_y = monitor_geom.y;
-
-        var initial_width = width;
-        var initial_height = height;
 
-        var target_x = outer_rect.x - offset_x;
-        var target_y = outer_rect.y - offset_y;
+        var target_x = outer_rect.x - monitor_geom.x;
+        var target_y = outer_rect.y - monitor_geom.y;
 
         active = false;
         update_hover_widgets (true);
@@ -297,35 +287,8 @@ public class Gala.WindowClone : Clutter.Actor {
         new GesturePropertyTransition (this, gesture_tracker, "shadow-opacity", (uint8) 255, (uint8) 0).start (with_gesture);
         new GesturePropertyTransition (window_icon, gesture_tracker, "opacity", 255u, 0u).start (with_gesture, () => {
             update_hover_widgets (false);
-        });
-
-        GestureTracker.OnUpdate on_animation_update = (percentage) => {
-            var width = GestureTracker.animation_value (initial_width, outer_rect.width, percentage);
-            var height = GestureTracker.animation_value (initial_height, outer_rect.height, percentage);
-            var scale = GestureTracker.animation_value (initial_scale, target_scale, percentage);
-
-            set_window_icon_position (width, height, scale, false);
-        };
-
-        GestureTracker.OnEnd on_animation_end = (percentage, cancel_action) => {
-            if (cancel_action) {
-                return;
-            }
-
             toggle_shadow (false);
-
-            window_icon.save_easing_state ();
-            window_icon.set_easing_mode (Clutter.AnimationMode.EASE_OUT_QUAD);
-            window_icon.set_easing_duration (AnimationsSettings.get_animation_duration (MultitaskingView.ANIMATION_DURATION));
-            set_window_icon_position (outer_rect.width, outer_rect.height, target_scale);
-            window_icon.restore_easing_state ();
-        };
-
-        if (gesture_tracker == null || !with_gesture || !AnimationsSettings.get_enable_animations ()) {
-            on_animation_end (1, false, 0);
-        } else {
-            gesture_tracker.connect_handlers (null, (owned) on_animation_update, (owned) on_animation_end);
-        }
+        });
     }
 
     /**
@@ -348,10 +311,7 @@ public class Gala.WindowClone : Clutter.Actor {
         float intial_x = from_window_position ? outer_rect.x - monitor_geom.x : x;
         float intial_y = from_window_position ? outer_rect.y - monitor_geom.y : y;
 
-        var scale = display.get_monitor_scale (display.get_monitor_index_for_rect (rect));
-
         update_hover_widgets (true);
-        set_window_icon_position (initial_width, initial_height, scale);
 
         new GesturePropertyTransition (this, gesture_tracker, "x", intial_x, (float) rect.x).start (with_gesture);
         new GesturePropertyTransition (this, gesture_tracker, "y", intial_y, (float) rect.y).start (with_gesture);
@@ -360,44 +320,18 @@ public class Gala.WindowClone : Clutter.Actor {
         new GesturePropertyTransition (this, gesture_tracker, "shadow-opacity", (uint8) 0, (uint8) 255).start (with_gesture);
         new GesturePropertyTransition (window_icon, gesture_tracker, "opacity", 0u, 255u).start (with_gesture, () => {
             update_hover_widgets (false);
-        });
-
-        GestureTracker.OnUpdate on_animation_update = (percentage) => {
-            var width = GestureTracker.animation_value (initial_width, rect.width, percentage);
-            var height = GestureTracker.animation_value (initial_height, rect.height, percentage);
-
-            set_window_icon_position (width, height, scale, false);
-        };
-
-        GestureTracker.OnEnd on_animation_end = (percentage, cancel_action) => {
-            if (cancel_action) {
-                return;
-            }
-
-            var duration = AnimationsSettings.get_animation_duration (MultitaskingView.ANIMATION_DURATION);
-
-            window_icon.save_easing_state ();
-            window_icon.set_easing_mode (Clutter.AnimationMode.EASE_OUT_QUAD);
-            window_icon.set_easing_duration (duration);
-            set_window_icon_position (rect.width, rect.height, scale);
-            window_icon.restore_easing_state ();
-
             toggle_shadow (true);
-        };
-
-        if (gesture_tracker == null || !with_gesture || !AnimationsSettings.get_enable_animations ()) {
-            on_animation_end (1, false, 0);
-        } else {
-            gesture_tracker.connect_handlers (null, (owned) on_animation_update, (owned) on_animation_end);
-        }
+        });
     }
 
     public override void allocate (Clutter.ActorBox box) {
         base.allocate (box);
 
         if (clone == null || (drag_action != null && drag_action.dragging)) {
+            warning ("RETURN");
             return;
         }
+        warning ("ALLOCATE");
 
         var input_rect = window.get_buffer_rect ();
         var outer_rect = window.get_frame_rect ();
@@ -448,6 +382,13 @@ public class Gala.WindowClone : Clutter.Actor {
 
         var window_title_alloc = InternalUtils.actor_box_from_rect (window_title_x, window_title_y, window_title_width, window_title_height);
         window_title.allocate (window_title_alloc);
+
+        var window_icon_size = InternalUtils.scale_to_int (WINDOW_ICON_SIZE, monitor_scale);
+        var window_icon_x = (box.get_width () - window_icon_size) / 2;
+        var window_icon_y = box.get_height () - (window_icon_size * 0.75f);
+
+        var window_icon_alloc = InternalUtils.actor_box_from_rect (window_icon_x, window_icon_y, window_icon_size, window_icon_size);
+        window_icon.allocate (window_icon_alloc);
     }
 
 #if HAS_MUTTER45
@@ -563,6 +504,9 @@ public class Gala.WindowClone : Clutter.Actor {
      * position we just freed is immediately filled by the WindowCloneContainer.
      */
     private Clutter.Actor drag_begin (float click_x, float click_y) {
+        var last_window_icon_x = window_icon.x;
+        var last_window_icon_y = window_icon.y;
+
         float abs_x, abs_y;
         float prev_parent_x, prev_parent_y;
 
@@ -594,6 +538,9 @@ public class Gala.WindowClone : Clutter.Actor {
 
         set_position (abs_x + prev_parent_x, abs_y + prev_parent_y);
 
+        // Set the last position so that it animates from there and not 0, 0
+        window_icon.set_position (last_window_icon_x, last_window_icon_y);
+
         window_icon.save_easing_state ();
         window_icon.set_easing_duration (duration);
         window_icon.set_easing_mode (Clutter.AnimationMode.EASE_IN_OUT_CUBIC);
@@ -742,7 +689,7 @@ public class Gala.WindowClone : Clutter.Actor {
      */
     private void drag_canceled () {
         get_parent ().remove_child (this);
-        prev_parent.insert_child_at_index (this, prev_index);
+        prev_parent.add_child (this); // Add above so that it is above while it animates back to its place
 
         var duration = AnimationsSettings.get_animation_duration (MultitaskingView.ANIMATION_DURATION);
 
@@ -756,28 +703,12 @@ public class Gala.WindowClone : Clutter.Actor {
 
         request_reposition ();
 
-        window_icon.save_easing_state ();
-        window_icon.set_easing_duration (duration);
-        window_icon.set_easing_mode (Clutter.AnimationMode.EASE_OUT_QUAD);
-
-        set_window_icon_position (slot.width, slot.height, monitor_scale_factor);
-        window_icon.restore_easing_state ();
-
         display.set_cursor (Meta.Cursor.DEFAULT);
-    }
-
-    private void set_window_icon_position (float window_width, float window_height, float scale_factor, bool aligned = true) {
-        var size = InternalUtils.scale_to_int (WINDOW_ICON_SIZE, scale_factor);
-        var x = (window_width - size) / 2;
-        var y = window_height - (size * 0.75f);
 
-        if (aligned) {
-            x = (int) Math.round (x);
-            y = (int) Math.round (y);
-        }
-
-        window_icon.set_size (size, size);
-        window_icon.set_position (x, y);
+        Timeout.add (duration, () => {
+            prev_parent.set_child_at_index (this, prev_index); // Set the correct index so that correct stacking order is kept
+            return Source.REMOVE;
+        });
     }
 
     private static bool is_close_button_on_left () {

src/WindowTracker.vala

@@ -52,7 +52,7 @@ public class Gala.WindowTracker : GLib.Object {
         return get_app_from_id (id);
     }
 
-    private static unowned Gala.App? get_app_from_pid (pid_t pid) {
+    private static unowned Gala.App? get_app_from_pid (int pid) {
         var running_apps = Gala.AppSystem.get_default ().get_running_apps ();
         foreach (unowned Gala.App app in running_apps) {
             var app_pids = app.get_pids ();

vapi/libmutter.vapi

@@ -1117,7 +1117,7 @@ namespace Meta {
 		public Meta.MaximizeFlags get_maximized ();
 		public int get_monitor ();
 		public unowned string? get_mutter_hints ();
-		public pid_t get_pid ();
+		public int get_pid ();
 		public unowned string get_role ();
 		public unowned string? get_sandboxed_app_id ();
 		public uint get_stable_sequence ();