Unify logic by introducing `scrollable::Scrollbar`

1 files changed, 61 insertions, 49 deletions

diff --git a/wgpu/src/renderer/widget/scrollable.rs b/wgpu/src/renderer/widget/scrollable.rs
index b83cee1b..6ef57185 100644
--- a/wgpu/src/renderer/widget/scrollable.rs
+++ b/wgpu/src/renderer/widget/scrollable.rs
@@ -5,30 +5,40 @@ const SCROLLBAR_WIDTH: u16 = 10;
 const SCROLLBAR_MARGIN: u16 = 2;
 
 impl scrollable::Renderer for Renderer {
-    fn scrollbar_bounds(bounds: Rectangle) -> Rectangle {
-        Rectangle {
-            x: bounds.x + bounds.width
-                - f32::from(SCROLLBAR_WIDTH + 2 * SCROLLBAR_MARGIN),
-            y: bounds.y,
-            width: f32::from(SCROLLBAR_WIDTH + 2 * SCROLLBAR_MARGIN),
-            height: bounds.height,
-        }
-    }
-
-    fn scroller_bounds(
+    fn scrollbar(
+        &self,
         bounds: Rectangle,
         content_bounds: Rectangle,
-        scrollbar_bounds: Rectangle,
         offset: u32,
-    ) -> Rectangle {
-        let ratio = bounds.height / content_bounds.height;
-        let scrollbar_height = bounds.height * ratio;
-        let y_offset = offset as f32 * ratio;
-        Rectangle {
-            x: scrollbar_bounds.x + f32::from(SCROLLBAR_MARGIN),
-            y: scrollbar_bounds.y + y_offset,
-            width: scrollbar_bounds.width - f32::from(2 * SCROLLBAR_MARGIN),
-            height: scrollbar_height,
+    ) -> Option<scrollable::Scrollbar> {
+        if content_bounds.height > bounds.height {
+            let scrollbar_bounds = Rectangle {
+                x: bounds.x + bounds.width
+                    - f32::from(SCROLLBAR_WIDTH + 2 * SCROLLBAR_MARGIN),
+                y: bounds.y,
+                width: f32::from(SCROLLBAR_WIDTH + 2 * SCROLLBAR_MARGIN),
+                height: bounds.height,
+            };
+
+            let ratio = bounds.height / content_bounds.height;
+            let scrollbar_height = bounds.height * ratio;
+            let y_offset = offset as f32 * ratio;
+
+            let scroller_bounds = Rectangle {
+                x: scrollbar_bounds.x + f32::from(SCROLLBAR_MARGIN),
+                y: scrollbar_bounds.y + y_offset,
+                width: scrollbar_bounds.width - f32::from(2 * SCROLLBAR_MARGIN),
+                height: scrollbar_height,
+            };
+
+            Some(scrollable::Scrollbar {
+                bounds: scrollbar_bounds,
+                scroller: scrollable::Scroller {
+                    bounds: scroller_bounds,
+                },
+            })
+        } else {
+            None
         }
     }
 
@@ -36,16 +46,13 @@ impl scrollable::Renderer for Renderer {
         &mut self,
         state: &scrollable::State,
         bounds: Rectangle,
-        content_bounds: Rectangle,
+        _content_bounds: Rectangle,
         is_mouse_over: bool,
         is_mouse_over_scrollbar: bool,
-        scrollbar_bounds: Rectangle,
-        scroller_bounds: Rectangle,
+        scrollbar: Option<scrollable::Scrollbar>,
         offset: u32,
         (content, mouse_cursor): Self::Output,
     ) -> Self::Output {
-        let is_content_overflowing = content_bounds.height > bounds.height;
-
         let clip = Primitive::Clip {
             bounds,
             offset: Vector::new(0, offset),
@@ -53,36 +60,41 @@ impl scrollable::Renderer for Renderer {
         };
 
         (
-            if is_content_overflowing
-                && (is_mouse_over || state.is_scroller_grabbed())
-            {
-                let scrollbar = Primitive::Quad {
-                    bounds: scroller_bounds,
-                    background: Background::Color([0.0, 0.0, 0.0, 0.7].into()),
-                    border_radius: 5,
-                };
-
-                if is_mouse_over_scrollbar || state.is_scroller_grabbed() {
-                    let scrollbar_background = Primitive::Quad {
-                        bounds: Rectangle {
-                            x: scrollbar_bounds.x + f32::from(SCROLLBAR_MARGIN),
-                            width: scrollbar_bounds.width
-                                - f32::from(2 * SCROLLBAR_MARGIN),
-                            ..scrollbar_bounds
-                        },
+            if let Some(scrollbar) = scrollbar {
+                if is_mouse_over || state.is_scroller_grabbed() {
+                    let scroller = Primitive::Quad {
+                        bounds: scrollbar.scroller.bounds,
                         background: Background::Color(
-                            [0.0, 0.0, 0.0, 0.3].into(),
+                            [0.0, 0.0, 0.0, 0.7].into(),
                         ),
                         border_radius: 5,
                     };
 
-                    Primitive::Group {
-                        primitives: vec![clip, scrollbar_background, scrollbar],
+                    if is_mouse_over_scrollbar || state.is_scroller_grabbed() {
+                        let scrollbar = Primitive::Quad {
+                            bounds: Rectangle {
+                                x: scrollbar.bounds.x
+                                    + f32::from(SCROLLBAR_MARGIN),
+                                width: scrollbar.bounds.width
+                                    - f32::from(2 * SCROLLBAR_MARGIN),
+                                ..scrollbar.bounds
+                            },
+                            background: Background::Color(
+                                [0.0, 0.0, 0.0, 0.3].into(),
+                            ),
+                            border_radius: 5,
+                        };
+
+                        Primitive::Group {
+                            primitives: vec![clip, scrollbar, scroller],
+                        }
+                    } else {
+                        Primitive::Group {
+                            primitives: vec![clip, scroller],
+                        }
                     }
                 } else {
-                    Primitive::Group {
-                        primitives: vec![clip, scrollbar],
-                    }
+                    clip
                 }
             } else {
                 clip