3 files changed, 57 insertions, 21 deletions

diff --git a/lib/astal/gtk3/src/widget/circularprogress.vala b/lib/astal/gtk3/src/widget/circularprogress.vala
index a3ecdf1..df1635d 100644
--- a/lib/astal/gtk3/src/widget/circularprogress.vala
+++ b/lib/astal/gtk3/src/widget/circularprogress.vala
@@ -44,26 +44,53 @@ public class Astal.CircularProgress : Gtk.Bin {
         set_css_name("circular-progress");
     }
 
+    public override Gtk.SizeRequestMode get_request_mode() {
+        if(get_child() != null) return get_child().get_request_mode();
+        return Gtk.SizeRequestMode.HEIGHT_FOR_WIDTH;
+    }
+
     public override void get_preferred_height(out int minh, out int nath) {
-        var val = get_style_context().get_property("min-height", Gtk.StateFlags.NORMAL);
-        if (val.get_int() <= 0) {
-            minh = 40;
-            nath = 40;
+        if(get_child() != null)  {
+          int minw, natw;
+          get_child().get_preferred_height(out minh, out nath);
+          get_child().get_preferred_width(out minw, out natw);
+
+          minh = int.max(minw, minh);
+          nath = int.max(natw, nath);
         }
+        var w_val = get_style_context().get_property("min-width", Gtk.StateFlags.NORMAL);
+        var h_val = get_style_context().get_property("min-height", Gtk.StateFlags.NORMAL);
+        minh = int.max(w_val.get_int(), minh);
+        nath = int.max(w_val.get_int(), nath);
+        minh = int.max(h_val.get_int(), minh);
+        nath = int.max(h_val.get_int(), nath);
+    }
 
-        minh = val.get_int();
-        nath = val.get_int();
+    public override void get_preferred_height_for_width(int width, out int minh, out int nath) {
+        minh = width;
+        nath = width;
     }
 
     public override void get_preferred_width(out int minw, out int natw) {
-        var val = get_style_context().get_property("min-width", Gtk.StateFlags.NORMAL);
-        if (val.get_int() <= 0) {
-            minw = 40;
-            natw = 40;
+        if(get_child() != null)  {
+          int minh, nath;
+          get_child().get_preferred_height(out minh, out nath);
+          get_child().get_preferred_width(out minw, out natw);
+
+          minw = int.max(minw, minh);
+          natw = int.max(natw, nath);
         }
+        var w_val = get_style_context().get_property("min-width", Gtk.StateFlags.NORMAL);
+        var h_val = get_style_context().get_property("min-height", Gtk.StateFlags.NORMAL);
+        minw = int.max(w_val.get_int(), minw);
+        natw = int.max(w_val.get_int(), natw);
+        minw = int.max(h_val.get_int(), minw);
+        natw = int.max(h_val.get_int(), natw);
+    }
 
-        minw = val.get_int();
-        natw = val.get_int();
+    public override void get_preferred_width_for_height(int height, out int minw, out int natw) {
+        minw = height;
+        natw = height;
     }
 
     private double to_radian(double percentage) {
@@ -115,6 +142,12 @@ public class Astal.CircularProgress : Gtk.Bin {
         Gtk.Allocation allocation;
         get_allocation(out allocation);
 
+        if (get_child() != null) {
+            get_child().size_allocate(allocation);
+            propagate_draw(get_child(), cr);
+        }
+
+
         var styles = get_style_context();
         var width = allocation.width;
         var height = allocation.height;
@@ -195,12 +228,6 @@ public class Astal.CircularProgress : Gtk.Bin {
             cr.arc(end_x, end_y, fg_stroke / 2, 0, 0 - 0.01);
             cr.fill();
         }
-
-        if (get_child() != null) {
-            get_child().size_allocate(allocation);
-            propagate_draw(get_child(), cr);
-        }
-
         return true;
     }
 }
diff --git a/lib/astal/io/process.vala b/lib/astal/io/process.vala
index cfd05b9..4b77aee 100644
--- a/lib/astal/io/process.vala
+++ b/lib/astal/io/process.vala
@@ -76,7 +76,7 @@ public class AstalIO.Process : Object {
      *
      * The first element of the vector is executed with the remaining elements as the argument list.
      */
-    public Process.subprocessv(string[] cmd) throws Error {
+    public Process(string[] cmd) throws Error {
         Object(argv: cmd);
         process = new Subprocess.newv(cmd,
             SubprocessFlags.STDIN_PIPE |
@@ -91,13 +91,22 @@ public class AstalIO.Process : Object {
     }
 
     /**
+     * Start a new subprocess with the given command.
+     *
+     * The first element of the vector is executed with the remaining elements as the argument list.
+     */
+    public static Process subprocessv(string[] cmd) throws Error {
+        return new Process(cmd);
+    }
+
+    /**
      * Start a new subprocess with the given command
      * which is parsed using [[email protected]_parse_argv].
      */
     public static Process subprocess(string cmd) throws Error {
         string[] argv;
         Shell.parse_argv(cmd, out argv);
-        return new Process.subprocessv(argv);
+        return Process.subprocessv(argv);
     }
 
     /**
diff --git a/lib/astal/io/variable.vala b/lib/astal/io/variable.vala
index 312a27a..e4105f8 100644
--- a/lib/astal/io/variable.vala
+++ b/lib/astal/io/variable.vala
@@ -172,7 +172,7 @@ public class AstalIO.Variable : VariableBase {
         return_if_fail(watch_proc == null);
         return_if_fail(watch_exec != null);
 
-        watch_proc = new Process.subprocessv(watch_exec);
+        watch_proc = Process.subprocessv(watch_exec);
         watch_proc.stdout.connect((str) => set_closure(str, watch_transform));
         watch_proc.stderr.connect((str) => this.error(str));
     }