Merge pull request #3 from Aylur/feat/circular_progress

fixes circular progress widgets

6 files changed, 65 insertions, 52 deletions

diff --git a/core/gjs/src/jsx/jsx-runtime.ts b/core/gjs/src/jsx/jsx-runtime.ts
index 70f098f..02fd39d 100644
--- a/core/gjs/src/jsx/jsx-runtime.ts
+++ b/core/gjs/src/jsx/jsx-runtime.ts
@@ -35,7 +35,7 @@ const ctors = {
     box: Widget.Box,
     button: Widget.Button,
     centerbox: Widget.CenterBox,
-    // TODO: circularprogress
+    circularprogress: Widget.CircularProgress,
     drawingarea: Widget.DrawingArea,
     entry: Widget.Entry,
     eventbox: Widget.EventBox,
@@ -63,7 +63,7 @@ declare global {
             box: Widget.BoxProps
             button: Widget.ButtonProps
             centerbox: Widget.CenterBoxProps
-            // TODO: circularprogress
+            circularprogress: Widget.CircularProgressProps,
             drawingarea: Widget.DrawingAreaProps
             entry: Widget.EntryProps
             eventbox: Widget.EventBoxProps
diff --git a/core/gjs/src/widgets.ts b/core/gjs/src/widgets.ts
index 82d4708..0a71d8b 100644
--- a/core/gjs/src/widgets.ts
+++ b/core/gjs/src/widgets.ts
@@ -26,7 +26,11 @@ export type CenterBox = Widget<Astal.CenterBox>
 export const CenterBox = astalify<typeof Astal.CenterBox, CenterBoxProps, "CenterBox">(Astal.CenterBox)
 export type CenterBoxProps = ConstructProps<Astal.CenterBox, Astal.CenterBox.ConstructorProps>
 
-// TODO: CircularProgress
+// CircularProgress
+export type CircularProgress = Widget<Astal.CircularProgress>
+export const CircularProgress = astalify<typeof Astal.CircularProgress, CircularProgressProps, "CircularProgress">(Astal.CircularProgress)
+export type CircularProgressProps = ConstructProps<Astal.CircularProgress, Astal.CircularProgress.ConstructorProps>
+
 
 // DrawingArea
 export type DrawingArea = Widget<Gtk.DrawingArea>
diff --git a/core/lua/astal/widget.lua b/core/lua/astal/widget.lua
index d2dadc6..e2bd612 100644
--- a/core/lua/astal/widget.lua
+++ b/core/lua/astal/widget.lua
@@ -209,7 +209,7 @@ local Widget = {
     Box = astalify(Astal.Box),
     Button = astalify(Astal.Button),
     CenterBox = astalify(Astal.CenterBox),
-    -- TODO: CircularProgress
+    CircularProgress = astalify(Astal.CircularProgress),
     DrawingArea = astalify(Gtk.DrawingArea),
     Entry = astalify(Gtk.Entry),
     EventBox = astalify(Astal.EventBox),
diff --git a/core/meson.build b/core/meson.build
index 4020a81..12b4f1d 100644
--- a/core/meson.build
+++ b/core/meson.build
@@ -15,8 +15,6 @@ prefix = get_option('prefix')
 libdir = get_option('prefix') / get_option('libdir')
 pkgdatadir = prefix / get_option('datadir') / 'astal'
 
-# math
-add_project_arguments(['-X', '-lm'], language: 'vala')
 
 assert(
   get_option('lib') or get_option('cli'),
diff --git a/core/src/meson.build b/core/src/meson.build
index 3e28d16..ade182e 100644
--- a/core/src/meson.build
+++ b/core/src/meson.build
@@ -14,7 +14,7 @@ config = configure_file(
   },
 )
 
-deps = [
+pkgconfig_deps = [
   dependency('glib-2.0'),
   dependency('gio-unix-2.0'),
   dependency('gobject-2.0'),
@@ -24,12 +24,14 @@ deps = [
   dependency('gtk-layer-shell-0'),
 ]
 
+deps = pkgconfig_deps + meson.get_compiler('c').find_library('m')
+
 sources = [
   config,
   'widget/box.vala',
   'widget/button.vala',
   'widget/centerbox.vala',
-  # 'widget/circularprogress.vala', # TODO: math lib -X -lm
+  'widget/circularprogress.vala',
   'widget/eventbox.vala',
   'widget/icon.vala',
   'widget/label.vala',
@@ -65,7 +67,7 @@ if get_option('lib')
     filebase: meson.project_name() + '-' + api_version,
     version: meson.project_version(),
     subdirs: meson.project_name(),
-    requires: deps,
+    requires: pkgconfig_deps,
     install_dir: libdir / 'pkgconfig',
     variables: {
       'gjs': pkgdatadir / 'gjs',
diff --git a/core/src/widget/circularprogress.vala b/core/src/widget/circularprogress.vala
index 9cd3e26..4e1410d 100644
--- a/core/src/widget/circularprogress.vala
+++ b/core/src/widget/circularprogress.vala
@@ -1,6 +1,5 @@
 namespace Astal {
 public class CircularProgress : Gtk.Bin {
-    public new Gtk.Widget child { get; set; }
     public double start_at { get; set; }
     public double end_at { get; set; }
     public double value { get; set; }
@@ -20,7 +19,7 @@ public class CircularProgress : Gtk.Bin {
         set_css_name("circular-progress");
     }
 
-    public new void get_preferred_height(out int minh, out int nath) {
+    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;
@@ -31,7 +30,7 @@ public class CircularProgress : Gtk.Bin {
         nath = val.get_int();
     }
 
-    public new void get_preferred_width(out int minw, out int natw) {
+    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;
@@ -42,12 +41,12 @@ public class CircularProgress : Gtk.Bin {
         natw = val.get_int();
     }
 
-    private double _to_radian(double percentage) {
+    private double to_radian(double percentage) {
         percentage = Math.floor(percentage * 100);
         return (percentage / 100) * (2 * Math.PI);
     }
 
-    private bool _is_full_circle(double start, double end, double epsilon = 1e-10) {
+    private bool is_full_circle(double start, double end, double epsilon = 1e-10) {
         // Ensure that start and end are between 0 and 1
         start = (start % 1 + 1) % 1;
         end = (end % 1 + 1) % 1;
@@ -56,40 +55,38 @@ public class CircularProgress : Gtk.Bin {
         return Math.fabs(start - end) <= epsilon;
     }
 
-    private double _map_arc_value_to_range(double start, double end, double value) {
+    private double scale_arc_value(double start, double end, double value) {
         // Ensure that start and end are between 0 and 1
         start = (start % 1 + 1) % 1;
         end = (end % 1 + 1) % 1;
 
         // Calculate the length of the arc
-        var arcLength = end - start;
-        if (arcLength < 0)
-            arcLength += 1; // Adjust for circular representation
+        var arc_length = end - start;
+        if (arc_length < 0)
+            arc_length += 1; // Adjust for circular representation
 
         // Calculate the position on the arc based on the percentage value
-        var position = start + (arcLength * value);
+        var scaled = arc_length + value;
 
         // Ensure the position is between 0 and 1
-        position = (position % 1 + 1) % 1;
-
-        return position;
+        return (scaled % 1 + 1) % 1;
     }
 
-    private double _min(double[] arr) {
+    private double min(double[] arr) {
         double min = arr[0];
         foreach(var i in arr)
             if (min > i) min = i;
         return min;
     }
 
-    private double _max(double[] arr) {
+    private double max(double[] arr) {
         double max = arr[0];
         foreach(var i in arr)
             if (max < i) max = i;
         return max;
     }
 
-    public new bool draw(Cairo.Context cr) {
+    public override bool draw(Cairo.Context cr) {
         Gtk.Allocation allocation;
         get_allocation(out allocation);
 
@@ -101,60 +98,72 @@ public class CircularProgress : Gtk.Bin {
         var fg = styles.get_color(Gtk.StateFlags.NORMAL);
         var bg = styles.get_background_color(Gtk.StateFlags.NORMAL);
 
-        var bg_stroke = thickness + _min({margin.bottom, margin.top, margin.left, margin.right});
+        var bg_stroke = thickness + min({margin.bottom, margin.top, margin.left, margin.right});
         var fg_stroke = thickness;
-        var radius = _min({width, height}) / 2.0 - _max({bg_stroke, fg_stroke}) / 2.0;
+        var radius = min({width, height}) / 2.0 - max({bg_stroke, fg_stroke}) / 2.0;
         var center_x = width / 2;
         var center_y = height / 2;
 
-        var start_background = _to_radian(this.start_at);
-        var end_background = _to_radian(this.end_at);
-        var ranged_value = this.value + this.start_at;
+        var start_background = to_radian(start_at);
+        var end_background = to_radian(end_at);
+        var ranged_value = value + start_at;
 
-        var is_circle = _is_full_circle(this.start_at, this.end_at);
+        var is_circle = is_full_circle(this.start_at, this.end_at);
 
         if (is_circle) {
-            // Redefine endDraw in radius to create an accurate full circle
+            // Redefine end_draw in radius to create an accurate full circle
             end_background = start_background + 2 * Math.PI;
+            ranged_value = to_radian(value);
         } else {
             // Range the value for the arc shape
-            ranged_value = _map_arc_value_to_range(
-                this.start_at,
-                this.end_at,
-                this.value
-            );
+            ranged_value = to_radian(scale_arc_value(
+                start_at,
+                end_at,
+                value
+            ));
         }
 
-        var to = _to_radian(ranged_value);
         double start_progress, end_progress;
 
-        if (this.inverted) {
-            start_progress = (2 * Math.PI - to) - start_background;
-            end_progress = (2 * Math.PI - start_background) - start_background;
+        if (inverted) {
+            start_progress = end_background - ranged_value;
+            end_progress = end_background;
         } else {
             start_progress = start_background;
-            end_progress = to;
+            end_progress = start_background + ranged_value;
         }
 
         // Draw background
         cr.set_source_rgba(bg.red, bg.green, bg.blue, bg.alpha);
         cr.arc(center_x, center_y, radius, start_background, end_background);
-
         cr.set_line_width(bg_stroke);
         cr.stroke();
 
+        // Draw rounded background ends
+        if (rounded) {
+            var start_x = center_x + Math.cos(start_background) * radius;
+            var start_y = center_y + Math.sin(start_background) * radius;
+            var end_x = center_x + Math.cos(end_background) * radius;
+            var end_y = center_y + Math.sin(end_background) * radius;
+            cr.set_line_width(0);
+            cr.arc(start_x, start_y, bg_stroke / 2, 0, 0 - 0.01);
+            cr.fill();
+            cr.arc(end_x, end_y, bg_stroke / 2, 0, 0 - 0.01);
+            cr.fill();
+        }
+
         // Draw progress
         cr.set_source_rgba(fg.red, fg.green, fg.blue, fg.alpha);
         cr.arc(center_x, center_y, radius, start_progress, end_progress);
         cr.set_line_width(fg_stroke);
         cr.stroke();
 
-        // Draw rounded ends
-        if (this.rounded) {
-            var start_x = center_x + Math.cos(start_background);
-            var start_y = center_y + Math.cos(start_background);
-            var end_x = center_x + Math.cos(to) * radius;
-            var end_y = center_y + Math.cos(to) * radius;
+        // Draw rounded progress ends
+        if (rounded) {
+            var start_x = center_x + Math.cos(start_progress) * radius;
+            var start_y = center_y + Math.sin(start_progress) * radius;
+            var end_x = center_x + Math.cos(end_progress) * radius;
+            var end_y = center_y + Math.sin(end_progress) * radius;
             cr.set_line_width(0);
             cr.arc(start_x, start_y, fg_stroke / 2, 0, 0 - 0.01);
             cr.fill();
@@ -162,9 +171,9 @@ public class CircularProgress : Gtk.Bin {
             cr.fill();
         }
 
-        if (this.child != null) {
-            this.child.size_allocate(allocation);
-            this.propagate_draw(this.child, cr);
+        if (get_child() != null) {
+            get_child().size_allocate(allocation);
+            propagate_draw(get_child(), cr);
         }
 
         return true;