BUGFIX: changing inner state with a collection wouldn't reload the collection

Author: togglebyte

anathema-runtime/src/runtime/mod.rs

@@ -270,6 +270,7 @@ impl<'rt, 'bp, G: GlobalEventHandler> Frame<'rt, 'bp, G> {
         match event {
             ComponentEvent::Noop => return,
             ComponentEvent::Stop => todo!(),
+
             // Component specific event
             ComponentEvent::Blur | ComponentEvent::Focus | ComponentEvent::Key(_) => {
                 let Some(Index {
@@ -395,6 +396,10 @@ impl<'rt, 'bp, G: GlobalEventHandler> Frame<'rt, 'bp, G> {
         #[cfg(feature = "profile")]
         puffin::profile_function!();
 
+        // TODO: let's keep some memory around to drain this into instead of allocating
+        // a new vector every time.
+        // E.g `self.deferred_components.drain_into(&mut self.deferred_buffer)`
+        // Nb: Add drain_into to DeferredComponents
         let commands = self.deferred_components.drain().collect::<Vec<_>>();
         for mut cmd in commands {
             // Blur the current component if the message is a `Focus` message

anathema-state/src/lib.rs

@@ -57,6 +57,18 @@ impl Hex {
     pub const GREEN: Self = Self { r: 0, g: 255, b: 0 };
     pub const RED: Self = Self { r: 255, g: 0, b: 0 };
     pub const WHITE: Self = Self { r: 255, g: 255, b: 255 };
+
+    pub const fn red_f32(&self) -> f32 {
+        self.r as f32 / u8::MAX as f32
+    }
+
+    pub const fn green_f32(&self) -> f32 {
+        self.g as f32 / u8::MAX as f32
+    }
+
+    pub const fn blue_f32(&self) -> f32 {
+        self.b as f32 / u8::MAX as f32
+    }
 }
 
 impl Display for Hex {

anathema-widgets/src/components/mod.rs

@@ -314,23 +314,23 @@ impl<'a> Event<'a> {
         self.external_ident
     }
 
-    /// Try to cast the event payload to a specific type
-    pub fn data<T: 'static>(&self) -> Option<&'a T> {
-        self.data.downcast_ref()
-    }
-
     /// Cast the event payload to a specific type
     ///
     /// # Panics
     ///
     /// This will panic if the type is incorrect
-    pub fn data_unchecked<T: 'static>(&self) -> &'a T {
+    pub fn data<T: 'static>(&self) -> &'a T {
         match self.data.downcast_ref() {
             Some(data) => data,
-            None => panic!("invalid type"),
+            None => panic!("invalid type when casting event data"),
         }
     }
 
+    /// Try to cast the event payload to a specific type
+    pub fn data_checked<T: 'static>(&self) -> Option<&'a T> {
+        self.data.downcast_ref()
+    }
+
     pub fn should_stop_propagation(&self) -> bool {
         self.stop_propagation
     }

anathema-widgets/src/nodes/loops.rs

@@ -76,8 +76,7 @@ impl<'bp> For<'bp> {
 
                 tree.relative_remove(&[*index as u16])
             }
-            Change::Dropped => tree.truncate_children(),
-            Change::Changed => {
+            Change::Dropped | Change::Changed => {
                 // If the collection has changed to a different collection
                 // then truncate the tree
                 self.collection.reload(attribute_storage);

examples/message-passing.rs

@@ -1,6 +1,5 @@
 use anathema::backend::tui::TuiBackend;
-use anathema::component::{Children, Component, MouseEvent};
-use anathema::prelude::Context;
+use anathema::component::{Children, Component, Context, MouseEvent};
 use anathema::runtime::Runtime;
 use anathema::state::{List, State, Value};
 use anathema::templates::Document;

src/lib.rs

@@ -17,14 +17,13 @@ pub mod prelude {
     pub use crate::backend::tui::TuiBackend;
     pub use crate::runtime::Runtime;
     pub use crate::templates::{ComponentBlueprintId, Document, SourceKind, ToSourceKind};
-    pub use crate::widgets::components::{Context, Event};
 }
 
 pub mod component {
     pub use crate::state::{Color, List, Map, Maybe, Nullable, State, Value};
     pub use crate::widgets::components::events::{
         ComponentEvent, KeyCode, KeyEvent, KeyState, MouseButton, MouseEvent, MouseState,
     };
-    pub use crate::widgets::components::{Component, ComponentId, Context, Emitter};
+    pub use crate::widgets::components::{Component, ComponentId, Context, Emitter, Event};
     pub use crate::widgets::query::Children;
 }