Move mesh::Style to triangle and reuse it in fill and stroke

graphics/src/widget/canvas/fill.rs

@@ -1,17 +1,15 @@
 //! Fill [crate::widget::canvas::Geometry] with a certain style.
+use crate::{Color, Gradient};
 
-use crate::gradient::Gradient;
-use crate::layer::mesh;
-use crate::widget::canvas::frame::Transform;
-use iced_native::Color;
+pub use crate::triangle::Style;
 
 /// The style used to fill geometry.
 #[derive(Debug, Clone)]
-pub struct Fill<'a> {
+pub struct Fill {
     /// The color or gradient of the fill.
     ///
     /// By default, it is set to [`FillStyle::Solid`] `BLACK`.
-    pub style: Style<'a>,
+    pub style: Style,
 
     /// The fill rule defines how to determine what is inside and what is
     /// outside of a shape.
@@ -24,17 +22,17 @@ pub struct Fill<'a> {
     pub rule: FillRule,
 }
 
-impl<'a> Default for Fill<'a> {
-    fn default() -> Fill<'a> {
-        Fill {
+impl Default for Fill {
+    fn default() -> Self {
+        Self {
             style: Style::Solid(Color::BLACK),
             rule: FillRule::NonZero,
         }
     }
 }
 
-impl<'a> From<Color> for Fill<'a> {
-    fn from(color: Color) -> Fill<'a> {
+impl From<Color> for Fill {
+    fn from(color: Color) -> Fill {
         Fill {
             style: Style::Solid(color),
             ..Fill::default()
@@ -42,8 +40,8 @@ impl<'a> From<Color> for Fill<'a> {
     }
 }
 
-impl<'a> From<&'a Gradient> for Fill<'a> {
-    fn from(gradient: &'a Gradient) -> Self {
+impl From<Gradient> for Fill {
+    fn from(gradient: Gradient) -> Self {
         Fill {
             style: Style::Gradient(gradient),
             ..Default::default()
@@ -51,27 +49,6 @@ impl<'a> From<&'a Gradient> for Fill<'a> {
     }
 }
 
-/// The style of a [`Fill`].
-#[derive(Debug, Clone)]
-pub enum Style<'a> {
-    /// A solid color
-    Solid(Color),
-    /// A color gradient
-    Gradient(&'a Gradient),
-}
-
-impl<'a> Style<'a> {
-    /// Converts a fill's [Style] to a [mesh::Style] for use in the renderer's shader.
-    pub(crate) fn as_mesh_style(&self, transform: &Transform) -> mesh::Style {
-        match self {
-            Style::Solid(color) => mesh::Style::Solid(*color),
-            Style::Gradient(gradient) => mesh::Style::Gradient(
-                transform.transform_gradient((*gradient).clone()),
-            ),
-        }
-    }
-}
-
 /// The fill rule defines how to determine what is inside and what is outside of
 /// a shape.
 ///

graphics/src/widget/canvas/frame.rs

@@ -4,7 +4,6 @@ use std::borrow::Cow;
 use iced_native::{Point, Rectangle, Size, Vector};
 
 use crate::gradient::Gradient;
-use crate::layer::mesh;
 use crate::triangle;
 use crate::triangle::Vertex2D;
 use crate::widget::canvas::{path, Fill, Geometry, Path, Stroke, Text};
@@ -26,7 +25,7 @@ pub struct Frame {
 }
 
 struct BufferStack {
-    stack: Vec<(tessellation::VertexBuffers<Vertex2D, u32>, mesh::Style)>,
+    stack: Vec<(tessellation::VertexBuffers<Vertex2D, u32>, triangle::Style)>,
 }
 
 impl BufferStack {
@@ -36,7 +35,7 @@ impl BufferStack {
 
     fn get(
         &mut self,
-        mesh_style: mesh::Style,
+        mesh_style: triangle::Style,
     ) -> tessellation::BuffersBuilder<'_, Vertex2D, u32, Vertex2DBuilder> {
         match self.stack.last_mut() {
             Some((_, current_style)) if current_style == &mesh_style => {}
@@ -74,6 +73,15 @@ impl Transform {
         point.y = transformed.y;
     }
 
+    fn transform_style(&self, style: triangle::Style) -> triangle::Style {
+        match style {
+            triangle::Style::Solid(color) => triangle::Style::Solid(color),
+            triangle::Style::Gradient(gradient) => {
+                triangle::Style::Gradient(self.transform_gradient(gradient))
+            }
+        }
+    }
+
     pub(crate) fn transform_gradient(
         &self,
         mut gradient: Gradient,
@@ -135,12 +143,12 @@ impl Frame {
 
     /// Draws the given [`Path`] on the [`Frame`] by filling it with the
     /// provided style.
-    pub fn fill<'a>(&mut self, path: &Path, fill: impl Into<Fill<'a>>) {
+    pub fn fill(&mut self, path: &Path, fill: impl Into<Fill>) {
         let Fill { style, rule } = fill.into();
 
         let mut buffer = self
             .buffers
-            .get(style.as_mesh_style(&self.transforms.current));
+            .get(self.transforms.current.transform_style(style));
 
         let options =
             tessellation::FillOptions::default().with_fill_rule(rule.into());
@@ -165,17 +173,17 @@ impl Frame {
 
     /// Draws an axis-aligned rectangle given its top-left corner coordinate and
     /// its `Size` on the [`Frame`] by filling it with the provided style.
-    pub fn fill_rectangle<'a>(
+    pub fn fill_rectangle(
         &mut self,
         top_left: Point,
         size: Size,
-        fill: impl Into<Fill<'a>>,
+        fill: impl Into<Fill>,
     ) {
         let Fill { style, rule } = fill.into();
 
         let mut buffer = self
             .buffers
-            .get(style.as_mesh_style(&self.transforms.current));
+            .get(self.transforms.current.transform_style(style));
 
         let top_left =
             self.transforms.current.raw.transform_point(
@@ -206,7 +214,7 @@ impl Frame {
 
         let mut buffer = self
             .buffers
-            .get(stroke.style.as_mesh_style(&self.transforms.current));
+            .get(self.transforms.current.transform_style(stroke.style));
 
         let mut options = tessellation::StrokeOptions::default();
         options.line_width = stroke.width;

graphics/src/widget/canvas/stroke.rs

@@ -1,8 +1,6 @@
 //! Create lines from a [crate::widget::canvas::Path] and assigns them various attributes/styles.
+pub use crate::triangle::Style;
 
-use crate::gradient::Gradient;
-use crate::layer::mesh;
-use crate::widget::canvas::frame::Transform;
 use iced_native::Color;
 
 /// The style of a stroke.
@@ -11,7 +9,7 @@ pub struct Stroke<'a> {
     /// The color or gradient of the stroke.
     ///
     /// By default, it is set to [`StrokeStyle::Solid`] `BLACK`.
-    pub style: Style<'a>,
+    pub style: Style,
     /// The distance between the two edges of the stroke.
     pub width: f32,
     /// The shape to be used at the end of open subpaths when they are stroked.
@@ -60,27 +58,6 @@ impl<'a> Default for Stroke<'a> {
     }
 }
 
-/// The style of a [`Stroke`].
-#[derive(Debug, Clone, Copy)]
-pub enum Style<'a> {
-    /// A solid color
-    Solid(Color),
-    /// A color gradient
-    Gradient(&'a Gradient),
-}
-
-impl<'a> Style<'a> {
-    /// Converts a fill's [Style] to a [mesh::Style] for use in the renderer's shader.
-    pub(crate) fn as_mesh_style(&self, transform: &Transform) -> mesh::Style {
-        match self {
-            Style::Solid(color) => mesh::Style::Solid(*color),
-            Style::Gradient(gradient) => mesh::Style::Gradient(
-                transform.transform_gradient((*gradient).clone()),
-            ),
-        }
-    }
-}
-
 /// The shape used at the end of open subpaths when they are stroked.
 #[derive(Debug, Clone, Copy)]
 pub enum LineCap {