From 21abef4cade9aaaa09d748d6d687d2cdc1e37db6 Mon Sep 17 00:00:00 2001
From: PolyMeilex <marynczakbartlomiej@gmail.com>
Date: Sat, 7 Dec 2024 20:14:25 +0100
Subject: [PATCH] nuon: Make widgets fully inmutable

---
 .../playing_scene/top_bar/widget/looper.rs    |  2 +-
 .../top_bar/widget/progress_bar.rs            |  2 +-
 .../top_bar/widget/speed_pill.rs              |  4 ---
 nuon/src/lib.rs                               | 29 +++++--------------
 nuon/src/widget/button.rs                     |  2 +-
 nuon/src/widget/column.rs                     |  4 ---
 nuon/src/widget/container.rs                  |  4 ---
 nuon/src/widget/row.rs                        |  4 ---
 nuon/src/widget/stack.rs                      |  4 ---
 nuon/src/widget/trilayout.rs                  |  4 ---
 10 files changed, 10 insertions(+), 49 deletions(-)

diff --git a/neothesia/src/scene/playing_scene/top_bar/widget/looper.rs b/neothesia/src/scene/playing_scene/top_bar/widget/looper.rs
index 1a52b27..446bca2 100644
--- a/neothesia/src/scene/playing_scene/top_bar/widget/looper.rs
+++ b/neothesia/src/scene/playing_scene/top_bar/widget/looper.rs
@@ -154,7 +154,7 @@ impl<MSG> Widget<MSG> for Looper<MSG> {
     }
 
     fn update(
-        &mut self,
+        &self,
         event: Event,
         layout: &Node,
         tree: &mut Tree<Self::State>,
diff --git a/neothesia/src/scene/playing_scene/top_bar/widget/progress_bar.rs b/neothesia/src/scene/playing_scene/top_bar/widget/progress_bar.rs
index 4b70ef8..65ceaa4 100644
--- a/neothesia/src/scene/playing_scene/top_bar/widget/progress_bar.rs
+++ b/neothesia/src/scene/playing_scene/top_bar/widget/progress_bar.rs
@@ -101,7 +101,7 @@ impl<MSG: Clone> Widget<MSG> for ProgressBar<MSG> {
     }
 
     fn update(
-        &mut self,
+        &self,
         event: Event,
         layout: &Node,
         tree: &mut Tree<Self::State>,
diff --git a/neothesia/src/scene/playing_scene/top_bar/widget/speed_pill.rs b/neothesia/src/scene/playing_scene/top_bar/widget/speed_pill.rs
index 607facd..afbb367 100644
--- a/neothesia/src/scene/playing_scene/top_bar/widget/speed_pill.rs
+++ b/neothesia/src/scene/playing_scene/top_bar/widget/speed_pill.rs
@@ -75,10 +75,6 @@ impl<MSG: Clone> Widget<MSG> for SpeedPill<MSG> {
         &self.children
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.children
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         let minus = self.children[0].as_widget().layout(
             tree.children[0].remap_mut(),
diff --git a/nuon/src/lib.rs b/nuon/src/lib.rs
index ea5e108..506fddc 100644
--- a/nuon/src/lib.rs
+++ b/nuon/src/lib.rs
@@ -103,20 +103,13 @@ pub trait WidgetAny<MSG> {
     fn state(&self) -> Box<dyn Any>;
 
     fn children(&self) -> &[Element<MSG>];
-    fn children_mut(&mut self) -> &mut [Element<MSG>];
     fn children_tree(&self) -> Vec<Tree>;
 
     fn diff(&self, tree: &mut Tree);
 
     fn layout(&self, tree: &mut Tree, avalilable: &ParentLayout, ctx: &LayoutCtx) -> Node;
     fn render(&self, renderer: &mut dyn Renderer, layout: &Node, tree: &Tree, ctx: &RenderCtx);
-    fn update(
-        &mut self,
-        event: input::Event,
-        layout: &Node,
-        tree: &mut Tree,
-        ctx: &mut UpdateCtx<MSG>,
-    );
+    fn update(&self, event: input::Event, layout: &Node, tree: &mut Tree, ctx: &mut UpdateCtx<MSG>);
 }
 
 impl<MSG, W: Widget<MSG>> WidgetAny<MSG> for W {
@@ -132,10 +125,6 @@ impl<MSG, W: Widget<MSG>> WidgetAny<MSG> for W {
         Widget::children(self)
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        Widget::children_mut(self)
-    }
-
     fn children_tree(&self) -> Vec<Tree> {
         Widget::children_tree(self)
     }
@@ -153,7 +142,7 @@ impl<MSG, W: Widget<MSG>> WidgetAny<MSG> for W {
     }
 
     fn update(
-        &mut self,
+        &self,
         event: input::Event,
         layout: &Node,
         tree: &mut Tree,
@@ -174,10 +163,6 @@ pub trait Widget<MSG> {
         &[]
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut []
-    }
-
     fn children_tree(&self) -> Vec<Tree> {
         self.children()
             .iter()
@@ -204,7 +189,7 @@ pub trait Widget<MSG> {
     }
 
     fn update(
-        &mut self,
+        &self,
         event: input::Event,
         layout: &Node,
         tree: &mut Tree<Self::State>,
@@ -236,20 +221,20 @@ pub fn default_render<MSG, W: Widget<MSG> + ?Sized>(
 }
 
 pub fn default_update<MSG, W: Widget<MSG> + ?Sized>(
-    this: &mut W,
+    this: &W,
     event: input::Event,
     layout: &Node,
     tree: &mut Tree<W::State>,
     ctx: &mut UpdateCtx<MSG>,
 ) {
     for ((ch, layout), tree) in this
-        .children_mut()
-        .iter_mut()
+        .children()
+        .iter()
         .zip(layout.children.iter())
         .zip(tree.children.iter_mut())
         .rev()
     {
-        ch.as_widget_mut().update(event.clone(), layout, tree, ctx);
+        ch.as_widget().update(event.clone(), layout, tree, ctx);
 
         if ctx.is_event_captured() {
             return;
diff --git a/nuon/src/widget/button.rs b/nuon/src/widget/button.rs
index a3ea2bf..ebb69be 100644
--- a/nuon/src/widget/button.rs
+++ b/nuon/src/widget/button.rs
@@ -140,7 +140,7 @@ impl<MSG: Clone> Widget<MSG> for Button<MSG> {
     }
 
     fn update(
-        &mut self,
+        &self,
         event: Event,
         layout: &Node,
         tree: &mut Tree<Self::State>,
diff --git a/nuon/src/widget/column.rs b/nuon/src/widget/column.rs
index 9e5c30b..6689e23 100644
--- a/nuon/src/widget/column.rs
+++ b/nuon/src/widget/column.rs
@@ -47,10 +47,6 @@ impl<MSG> Widget<MSG> for Column<MSG> {
         &self.children
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.children
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         column_layout(self, tree, parent, ctx, self.gap)
     }
diff --git a/nuon/src/widget/container.rs b/nuon/src/widget/container.rs
index 9a5c17c..3ce267f 100644
--- a/nuon/src/widget/container.rs
+++ b/nuon/src/widget/container.rs
@@ -65,10 +65,6 @@ impl<MSG> Widget<MSG> for Container<MSG> {
         &self.child
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.child
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         let parent = &ParentLayout {
             x: parent.x + self.x,
diff --git a/nuon/src/widget/row.rs b/nuon/src/widget/row.rs
index edf7a07..aea6322 100644
--- a/nuon/src/widget/row.rs
+++ b/nuon/src/widget/row.rs
@@ -47,10 +47,6 @@ impl<MSG> Widget<MSG> for Row<MSG> {
         &self.children
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.children
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         row_layout(self, tree, parent, ctx, self.gap)
     }
diff --git a/nuon/src/widget/stack.rs b/nuon/src/widget/stack.rs
index a1b1e1a..bd69e93 100644
--- a/nuon/src/widget/stack.rs
+++ b/nuon/src/widget/stack.rs
@@ -40,10 +40,6 @@ impl<MSG> Widget<MSG> for Stack<MSG> {
         &self.children
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.children
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         stack_layout(self, tree, parent, ctx)
     }
diff --git a/nuon/src/widget/trilayout.rs b/nuon/src/widget/trilayout.rs
index 8534a64..37d0db7 100644
--- a/nuon/src/widget/trilayout.rs
+++ b/nuon/src/widget/trilayout.rs
@@ -54,10 +54,6 @@ impl<MSG> Widget<MSG> for TriLayout<MSG> {
         &self.children
     }
 
-    fn children_mut(&mut self) -> &mut [Element<MSG>] {
-        &mut self.children
-    }
-
     fn layout(&self, tree: &mut Tree<Self::State>, parent: &ParentLayout, ctx: &LayoutCtx) -> Node {
         let start = self.children[0].as_widget().layout(
             &mut tree.children[0],