Merge `rect` and `box` 🦚

3 files changed, 44 insertions, 87 deletions

diff --git a/src/layout/background.rs b/src/layout/background.rs
new file mode 100644
index 00000000..94d5bf45
--- /dev/null
+++ b/src/layout/background.rs
@@ -0,0 +1,37 @@
+use super::*;
+
+/// A node that represents a rectangular box.
+#[derive(Debug, Clone, PartialEq)]
+pub struct NodeBackground {
+    /// The background fill.
+    pub fill: Fill,
+    /// The child node whose sides to pad.
+    pub child: NodeFixed,
+}
+
+impl Layout for NodeBackground {
+    fn layout(&self, ctx: &mut LayoutContext, areas: &Areas) -> Layouted {
+        let mut layouted = self.child.layout(ctx, areas);
+
+        if let Some(first) = layouted.frames_mut().first_mut() {
+            first.elements.insert(
+                0,
+                (
+                    Point::ZERO,
+                    Element::Geometry(Geometry {
+                        shape: Shape::Rect(first.size),
+                        fill: self.fill.clone(),
+                    }),
+                ),
+            )
+        }
+
+        layouted
+    }
+}
+
+impl From<NodeBackground> for NodeAny {
+    fn from(background: NodeBackground) -> Self {
+        Self::new(background)
+    }
+}
diff --git a/src/layout/mod.rs b/src/layout/mod.rs
index 1bb4419d..0c7a6a68 100644
--- a/src/layout/mod.rs
+++ b/src/layout/mod.rs
@@ -4,7 +4,7 @@ mod fixed;
 mod node;
 mod pad;
 mod par;
-mod rect;
+mod background;
 mod spacing;
 mod stack;
 mod text;
@@ -18,7 +18,7 @@ pub use fixed::*;
 pub use node::*;
 pub use pad::*;
 pub use par::*;
-pub use rect::*;
+pub use background::*;
 pub use spacing::*;
 pub use stack::*;
 pub use text::*;
@@ -236,7 +236,6 @@ pub enum Fill {
     Color(Color),
     /// The fill is an image.
     Image(Image),
-    // Gradient(Gradient),
 }
 
 /// A shape with some kind of fill.
@@ -245,10 +244,10 @@ pub struct Geometry {
     /// The shape to draw.
     pub shape: Shape,
     /// How the shape looks on the inside.
-    ///
-    /// **TODO:** This could be made into a Vec<Fill> or something such that
-    ///           the user can compose multiple fills with alpha values less
-    ///           than one to achieve cool effects.
+    //
+    //  TODO: This could be made into a Vec<Fill> or something such that
+    //        the user can compose multiple fills with alpha values less
+    //        than one to achieve cool effects.
     pub fill: Fill,
 }
 
@@ -256,15 +255,7 @@ pub struct Geometry {
 #[derive(Debug, Clone, PartialEq)]
 pub enum Shape {
     /// A rectangle.
-    Rect(Rect),
-    // Ellipse(Ellipse),
-}
-
-/// An rectangle.
-#[derive(Debug, Clone, PartialEq)]
-pub struct Rect {
-    /// The dimensions of the rectangle.
-    pub size: Size,
+    Rect(Size),
 }
 
 /// An image element.
diff --git a/src/layout/rect.rs b/src/layout/rect.rs
deleted file mode 100644
index ab8e3b07..00000000
--- a/src/layout/rect.rs
+++ /dev/null
@@ -1,71 +0,0 @@
-use std::cmp::Ordering;
-
-use super::*;
-use crate::{color::RgbaColor, geom::Linear};
-
-/// A node that represents a rectangular box.
-#[derive(Debug, Clone, PartialEq)]
-pub struct NodeRect {
-    /// The fixed width, if any.
-    pub width: Option<Linear>,
-    /// The fixed height, if any.
-    pub height: Option<Linear>,
-    /// The background color.
-    pub color: Option<Color>,
-    /// The child node whose sides to pad.
-    pub child: Node,
-}
-
-impl Layout for NodeRect {
-    fn layout(&self, ctx: &mut LayoutContext, areas: &Areas) -> Layouted {
-        let Areas { current, full, .. } = areas;
-
-        let height_opt = self.height.map(|h| h.resolve(full.height));
-        let mut size = Size::new(
-            self.width.map(|w| w.resolve(full.width)).unwrap_or(current.width),
-            height_opt.unwrap_or(current.height),
-        );
-
-        let areas = Areas::once(size);
-        let mut layouted = self.child.layout(ctx, &areas);
-
-        // If the children have some height, apply that,
-        // otherwise fall back to zero or the height property.
-        if let Some(max) = layouted
-            .frames()
-            .iter()
-            .map(|f| f.size.height)
-            .max_by(|x, y| x.partial_cmp(y).unwrap_or(Ordering::Equal))
-        {
-            size.height = max;
-        } else {
-            size.height = height_opt.unwrap_or(Length::ZERO)
-        }
-
-        if let Some(first) = layouted.frames_mut().first_mut() {
-            first.elements.insert(
-                0,
-                (
-                    Point::ZERO,
-                    Element::Geometry(Geometry {
-                        shape: Shape::Rect(Rect { size }),
-                        fill: Fill::Color(self.color.unwrap_or(Color::Rgba(RgbaColor {
-                            r: 255,
-                            g: 255,
-                            b: 255,
-                            a: 0,
-                        }))),
-                    }),
-                ),
-            )
-        }
-
-        layouted
-    }
-}
-
-impl From<NodeRect> for NodeAny {
-    fn from(pad: NodeRect) -> Self {
-        Self::new(pad)
-    }
-}