From 0eaaeaa517a00765045de155bb1de01c2d8f553f Mon Sep 17 00:00:00 2001
From: Héctor Ramón Jiménez <hector@hecrj.dev>
Date: Mon, 12 Feb 2024 19:24:09 +0100
Subject: Simplify `scrollable` styling API

---
 style/src/scrollable.rs | 55 ++++++++++++++++---------------------------------
 1 file changed, 18 insertions(+), 37 deletions(-)

(limited to 'style/src/scrollable.rs')

diff --git a/style/src/scrollable.rs b/style/src/scrollable.rs
index 3daa8886..d2348510 100644
--- a/style/src/scrollable.rs
+++ b/style/src/scrollable.rs
@@ -1,6 +1,18 @@
 //! Change the appearance of a scrollable.
+use crate::container;
 use crate::core::{Background, Border, Color};
 
+/// The appearance of a scrolable.
+#[derive(Debug, Clone, Copy)]
+pub struct Appearance {
+    /// The [`container::Appearance`] of a scrollable.
+    pub container: container::Appearance,
+    /// The [`Scrollbar`] appearance.
+    pub scrollbar: Scrollbar,
+    /// The [`Background`] of the gap between a horizontal and vertical scrollbar.
+    pub gap: Option<Background>,
+}
+
 /// The appearance of the scrollbar of a scrollable.
 #[derive(Debug, Clone, Copy)]
 pub struct Scrollbar {
@@ -21,54 +33,23 @@ pub struct Scroller {
     pub border: Border,
 }
 
-/// The appearance of a scrolable.
-#[derive(Debug, Clone, Copy)]
-pub struct Appearance {
-    /// The [`Background`] of a scrollable.
-    pub background: Option<Background>,
-    /// The [`Background`] of the gap between a horizontal and vertical scrollbar.
-    pub gap: Option<Background>,
-}
-
 /// A set of rules that dictate the style of a scrollable.
 pub trait StyleSheet {
     /// The supported style of the [`StyleSheet`].
     type Style: Default;
 
-    /// Produces the style of the scrollable container.
-    fn appearance(&self, style: &Self::Style) -> Appearance;
-
-    /// Produces the style of an active scrollbar.
-    fn active(&self, style: &Self::Style) -> Scrollbar;
+    /// Produces the [`Appearance`] of an active scrollable.
+    fn active(&self, style: &Self::Style) -> Appearance;
 
-    /// Produces the style of a scrollbar when the scrollable is being hovered.
+    /// Produces the [`Appearance`] of a scrollable when it is being hovered.
     fn hovered(
         &self,
         style: &Self::Style,
         is_mouse_over_scrollbar: bool,
-    ) -> Scrollbar;
+    ) -> Appearance;
 
-    /// Produces the style of a scrollbar that is being dragged.
-    fn dragging(&self, style: &Self::Style) -> Scrollbar {
+    /// Produces the [`Appearance`] of a scrollable when it is being dragged.
+    fn dragging(&self, style: &Self::Style) -> Appearance {
         self.hovered(style, true)
     }
-
-    /// Produces the style of an active horizontal scrollbar.
-    fn active_horizontal(&self, style: &Self::Style) -> Scrollbar {
-        self.active(style)
-    }
-
-    /// Produces the style of a horizontal scrollbar when the scrollable is being hovered.
-    fn hovered_horizontal(
-        &self,
-        style: &Self::Style,
-        is_mouse_over_scrollbar: bool,
-    ) -> Scrollbar {
-        self.hovered(style, is_mouse_over_scrollbar)
-    }
-
-    /// Produces the style of a horizontal scrollbar that is being dragged.
-    fn dragging_horizontal(&self, style: &Self::Style) -> Scrollbar {
-        self.hovered_horizontal(style, true)
-    }
 }
-- 
cgit