CloseButton: make monitor scale settable

Author: lenemter

lib/CloseButton.vala

@@ -8,16 +8,23 @@ public class Gala.CloseButton : Clutter.Actor {
     private static Gee.HashMap<int, Gdk.Pixbuf?> close_pixbufs;
 
     public signal void triggered (uint32 timestamp);
-    public float scale { get; construct set; }
+
+    private float _monitor_scale = 1.0f;
+    public float monitor_scale {
+        get {
+            return _monitor_scale;
+        }
+        set {
+            _monitor_scale = value;
+
+            load_pixbuf ();
+        }
+    }
 
     // used to avoid changing hitbox of the button
     private Clutter.Actor pixbuf_actor;
     private bool is_pressed = false;
 
-    public CloseButton (float scale) {
-        Object (scale: scale);
-    }
-
     static construct {
         close_pixbufs = new Gee.HashMap<int, Gdk.Pixbuf?> ();
     }
@@ -30,23 +37,23 @@ public class Gala.CloseButton : Clutter.Actor {
         };
         add_child (pixbuf_actor);
 
-        var pixbuf = get_close_button_pixbuf (scale);
+        load_pixbuf ();
+    }
+
+    private void load_pixbuf () {
+        var pixbuf = get_close_button_pixbuf (monitor_scale);
         if (pixbuf != null) {
-            try {
-                var image = new Gala.Image.from_pixbuf (pixbuf);
-                pixbuf_actor.set_content (image);
-                pixbuf_actor.set_size (pixbuf.width, pixbuf.height);
-                set_size (pixbuf.width, pixbuf.height);
-            } catch (Error e) {
-                create_error_texture ();
-            }
+            var image = new Gala.Image.from_pixbuf (pixbuf);
+            pixbuf_actor.set_content (image);
+            pixbuf_actor.set_size (pixbuf.width, pixbuf.height);
+            set_size (pixbuf.width, pixbuf.height);
         } else {
             create_error_texture ();
         }
     }
 
-    private static Gdk.Pixbuf? get_close_button_pixbuf (float scale) {
-        var height = Utils.scale_to_int (36, scale);
+    private static Gdk.Pixbuf? get_close_button_pixbuf (float monitor_scale) {
+        var height = Utils.scale_to_int (36, monitor_scale);
 
         if (close_pixbufs[height] == null) {
             try {
@@ -71,7 +78,7 @@ public class Gala.CloseButton : Clutter.Actor {
         // works as good as some weird fallback-image-failed-to-load pixbuf
         critical ("Could not create close button");
 
-        var size = Utils.scale_to_int (36, scale);
+        var size = Utils.scale_to_int (36, monitor_scale);
         pixbuf_actor.set_size (size, size);
         pixbuf_actor.background_color = { 255, 0, 0, 255 };
     }

plugins/pip/PopupWindow.vala

@@ -97,7 +97,7 @@ public class Gala.Plugins.PIP.PopupWindow : Clutter.Actor {
 
         set_position (x_position, y_position);
 
-        close_button = new Gala.CloseButton (scale) {
+        close_button = new Gala.CloseButton () {
             opacity = 0
         };
         // TODO: Check if close button should be on the right

src/Widgets/MultitaskingView/WindowClone.vala

@@ -136,9 +136,17 @@ public class Gala.WindowClone : ActorTarget, RootTarget {
 
         window_title = new Tooltip ();
 
+        close_button = new Gala.CloseButton () {
+            opacity = 0
+        };
+        bind_property ("monitor-scale", close_button, "monitor-scale", SYNC_CREATE);
+        close_button.triggered.connect (close_window);
+        close_button.notify["has-pointer"].connect (() => update_hover_widgets ());
+
         add_child (active_shape);
         add_child (clone_container);
         add_child (window_title);
+        add_child (close_button);
 
         notify["monitor-scale"].connect (reallocate);
         reallocate ();
@@ -163,19 +171,12 @@ public class Gala.WindowClone : ActorTarget, RootTarget {
     }
 
     private void reallocate () {
-        close_button = new Gala.CloseButton (monitor_scale) {
-            opacity = 0
-        };
-        close_button.triggered.connect (close_window);
-        close_button.notify["has-pointer"].connect (() => update_hover_widgets ());
-
         window_icon = new WindowIcon (window, WINDOW_ICON_SIZE, (int)Math.round (monitor_scale)) {
             visible = !overview_mode
         };
         window_icon.opacity = 0;
         window_icon.set_pivot_point (0.5f, 0.5f);
 
-        add_child (close_button);
         add_child (window_icon);
 
         set_child_below_sibling (window_icon, window_title);