CHANGELOG.md

@@ -1,22 +1,5 @@
 # CHANGELOG
 
-## [0.4.0]
-
-## Changed
-
-- Updated to Bevy 0.16
-
-## [0.3.2]
-
-## Added
-
-- `scroll_view_node` function with default values for `ScrollView` node
-- documentation to the code 
-
-## Fixed
-
-- `ScrollableContent` default Node values fix
-
 ## [0.3.1]
 
 ## Fixed

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "bevy_simple_scroll_view"
-version = "0.4.0"
+version = "0.3.1"
 edition = "2021"
 exclude = [".github/", "wasm/", "record.gif"]
 categories = ["game-development", "gui"]
@@ -11,17 +11,14 @@ license = "MIT OR Apache-2.0"
 description = "Simple to use plugin implementing ScrollView into Bevy engine."
 
 [dependencies.bevy]
-version = "0.16"
+version = "0.15"
 default-features = false
 features = ["bevy_ui", "bevy_asset", "bevy_text"]
 
 [dev-dependencies.bevy]
-version = "0.16"
+version = "0.15"
 default-features = true
 
 [features]
 default = []
-extra_logs = ["bevy/bevy_log"]
+extra_logs = []
-
-[lints.rust]
-missing_docs = "warn"

README.md

@@ -36,7 +36,6 @@ Please keep PRs small and scoped to a single feature or fix.
 
 Bevy version | crate version
 --- | ---
-0.16 | 0.4
 0.15 | 0.3
 0.14 | 0.2
 0.13 | 0.1

examples/simple.rs

@@ -1,84 +1,135 @@
-#![allow(missing_docs)]
-
-use bevy::picking::events::{Pointer, Released};
 use bevy::prelude::*;
 use bevy_simple_scroll_view::*;
 
-const BG_COLOR: BackgroundColor = BackgroundColor(Color::srgb(0.168, 0.168, 0.168));
+const CLR_1: Color = Color::srgb(0.168, 0.168, 0.168);
-const BG_COLOR_2: BackgroundColor = BackgroundColor(Color::srgb(0.109, 0.109, 0.109));
+const CLR_2: Color = Color::srgb(0.109, 0.109, 0.109);
 const CLR_3: Color = Color::srgb(0.569, 0.592, 0.647);
-const TEXT_COLOR: TextColor = TextColor(Color::srgb(0.902, 0.4, 0.004));
+const CLR_4: Color = Color::srgb(0.902, 0.4, 0.004);
 
 fn main() {
     App::new()
         .add_plugins((DefaultPlugins, ScrollViewPlugin))
-        .add_systems(Startup, setup)
+        .add_systems(Startup, prepare)
+        .add_systems(Update, reset_scroll)
         .run();
 }
 
-fn setup(mut commands: Commands) {
+fn prepare(mut commands: Commands) {
-    let base_node = Node {
-        align_items: AlignItems::Center,
-        justify_content: JustifyContent::Center,
-        min_width: Val::Px(200.0),
-        margin: UiRect::all(Val::Px(10.0)),
-        border: UiRect::all(Val::Px(5.0)),
-        padding: UiRect::all(Val::Px(15.0)),
-        ..default()
-    };
     commands.spawn(Camera2d);
     commands
         .spawn((
-            BG_COLOR,
+            BackgroundColor(CLR_1),
             Node {
                 width: Val::Percent(100.0),
                 height: Val::Percent(100.0),
+                padding: UiRect::all(Val::Px(15.0)),
                 ..default()
             },
         ))
         .with_children(|p| {
             p.spawn(Node {
                 width: Val::Percent(20.0),
-                margin: UiRect::all(Val::Px(15.0)),
+                margin: UiRect::all(Val::Px(10.0)),
                 flex_direction: FlexDirection::Column,
                 ..default()
             })
             .with_children(|p| {
-                p.spawn((Text::new("Scroll to:"), TEXT_COLOR));
+                for btn_action in [ScrollButton::MoveToTop, ScrollButton::MoveToBottom] {
-                let btn = (base_node.clone(), BG_COLOR_2, Button);
+                    p.spawn((
-                p.spawn(btn.clone())
+                        Node {
-                    .observe(scroll_to_top)
+                            margin: UiRect::all(Val::Px(15.0)),
-                    .with_child((Text::new("top"), TEXT_COLOR));
+                            padding: UiRect::all(Val::Px(15.0)),
-                p.spawn(btn)
+                            max_height: Val::Px(100.0),
-                    .observe(scroll_to_bottom)
+                            border: UiRect::all(Val::Px(3.0)),
-                    .with_child((Text::new("bottom"), TEXT_COLOR));
+                            align_items: AlignItems::Center,
+                            ..default()
+                        },
+                        BackgroundColor(CLR_2),
+                        BorderColor(CLR_4),
+                        Button,
+                        btn_action,
+                    ))
+                    .with_children(|p| {
+                        p.spawn((
+                            Text::new(format!("{:#?}", btn_action)),
+                            TextFont {
+                                font_size: 25.0,
+                                ..default()
+                            },
+                            TextColor(CLR_4),
+                        ));
+                    });
+                }
             });
             p.spawn((
                 Node {
                     width: Val::Percent(80.0),
                     margin: UiRect::all(Val::Px(15.0)),
-                    ..scroll_view_node()
+                    ..default()
                 },
-                BG_COLOR_2,
+                BackgroundColor(CLR_2),
                 ScrollView::default(),
             ))
             .with_children(|p| {
-                p.spawn(ScrollableContent::default())
+                p.spawn((
+                    Node {
+                        flex_direction: bevy::ui::FlexDirection::Column,
+                        width: Val::Percent(100.0),
+                        ..default()
+                    },
+                    ScrollableContent::default(),
+                ))
                 .with_children(|scroll_area| {
-                        for i in 1..21 {
+                    for i in 0..21 {
                         scroll_area
-                                .spawn((base_node.clone(), BorderColor(CLR_3)))
+                            .spawn((
-                                .with_child((Text::new(format!("Nr {} out of 20", i)), TEXT_COLOR));
+                                Node {
+                                    min_width: Val::Px(200.0),
+                                    margin: UiRect::all(Val::Px(15.0)),
+                                    border: UiRect::all(Val::Px(5.0)),
+                                    padding: UiRect::all(Val::Px(30.0)),
+                                    ..default()
+                                },
+                                BorderColor(CLR_3),
+                            ))
+                            .with_children(|p| {
+                                p.spawn((
+                                    Text::new(format!("Nr {}", i)),
+                                    TextFont {
+                                        font_size: 25.0,
+                                        ..default()
+                                    },
+                                    TextColor(CLR_3),
+                                    TextLayout::new_with_justify(JustifyText::Center),
+                                ));
+                            });
                     }
                 });
             });
         });
 }
 
-fn scroll_to_top(_t: Trigger<Pointer<Released>>, mut scroll: Single<&mut ScrollableContent>) {
+#[derive(Component, PartialEq, Debug, Clone, Copy)]
-    scroll.scroll_to_top();
+#[require(Button)]
+enum ScrollButton {
+    MoveToTop,
+    MoveToBottom,
 }
 
-fn scroll_to_bottom(_t: Trigger<Pointer<Released>>, mut scroll: Single<&mut ScrollableContent>) {
+fn reset_scroll(
-    scroll.scroll_to_bottom();
+    q: Query<(&Interaction, &ScrollButton), Changed<Interaction>>,
+    mut scrolls_q: Query<&mut ScrollableContent>,
+) {
+    let Ok(mut scroll) = scrolls_q.get_single_mut() else {
+        return;
+    };
+    for (interaction, action) in q.iter() {
+        if interaction != &Interaction::Pressed {
+            continue;
+        }
+        match action {
+            ScrollButton::MoveToTop => scroll.scroll_to_top(),
+            ScrollButton::MoveToBottom => scroll.scroll_to_bottom(),
+        }
+    }
 }

src/lib.rs

@@ -39,7 +39,7 @@ impl Plugin for ScrollViewPlugin {
 
 /// Root component of scroll, it should have clipped style.
 #[derive(Component, Debug, Reflect)]
-#[require(Interaction, Node = scroll_view_node())]
+#[require(Interaction, Node)]
 pub struct ScrollView {
     /// Field which control speed of the scrolling.
     /// Could be negative number to implement invert scroll
@@ -57,40 +57,13 @@ impl Default for ScrollView {
 /// Component containing offset value of the scroll container to the parent.
 /// It is possible to update the field `pos_y` manually to move scrollview to desired location.
 #[derive(Component, Debug, Reflect, Default)]
-#[require(Node = scroll_content_node())]
+#[require(Node(scroll_view_node))]
 pub struct ScrollableContent {
     /// Scroll container offset to the `ScrollView`.
     pub pos_y: f32,
-    /// Maximum value for the scroll. It is updated automatically based on the size of the children nodes.
     pub max_scroll: f32,
 }
 
-impl ScrollableContent {
-    /// Scrolls to the top of the scroll view.
-    pub fn scroll_to_top(&mut self) {
-        self.pos_y = 0.0;
-    }
-    /// Scrolls to the bottom of the scroll view.
-    pub fn scroll_to_bottom(&mut self) {
-        self.pos_y = -self.max_scroll;
-    }
-
-    /// Scrolls by a specified amount.
-    ///
-    /// # Parameters
-    /// - `value`: The amount to scroll vertically. Positive values scroll down,
-    ///   and negative values scroll up.
-    ///
-    /// Ensures the new position is clamped between the valid scroll range.
-    pub fn scroll_by(&mut self, value: f32) {
-        self.pos_y += value;
-        self.pos_y = self.pos_y.clamp(-self.max_scroll, 0.);
-    }
-}
-
-/// Creates a default scroll view node.
-///
-/// This function defines the visual and layout properties of a scrollable container.
 pub fn scroll_view_node() -> Node {
     Node {
         overflow: Overflow::clip(),
@@ -101,19 +74,19 @@ pub fn scroll_view_node() -> Node {
     }
 }
 
-/// Creates a default scroll content node.
+impl ScrollableContent {
-pub fn scroll_content_node() -> Node {
+    pub fn scroll_to_top(&mut self) {
-    Node {
+        self.pos_y = 0.0;
-        flex_direction: bevy::ui::FlexDirection::Column,
+    }
-        width: Val::Percent(100.0),
+    pub fn scroll_to_bottom(&mut self) {
-        ..default()
+        self.pos_y = -self.max_scroll;
+    }
+    pub fn scroll_by(&mut self, value: f32) {
+        self.pos_y += value;
+        self.pos_y = self.pos_y.clamp(-self.max_scroll, 0.);
     }
 }
 
-/// Applies the default scroll view style to newly added `ScrollView` components.
-///
-/// This function updates the style of all new `ScrollView` nodes with the default
-/// properties defined in `scroll_view_node`.
 pub fn create_scroll_view(mut q: Query<&mut Node, Added<ScrollView>>) {
     let Node {
         overflow,
@@ -140,7 +113,7 @@ fn input_mouse_pressed_move(
             if interaction != Interaction::Pressed {
                 continue;
             }
-            for child in children.iter() {
+            for &child in children.iter() {
                 let Ok(mut scroll) = content_q.get_mut(child) else {
                     continue;
                 };
@@ -156,7 +129,7 @@ fn update_size(
 ) {
     for (children, scroll_view_node) in q.iter_mut() {
         let container_height = scroll_view_node.size().y * scroll_view_node.inverse_scale_factor();
-        for child in children.iter() {
+        for &child in children.iter() {
             let Ok((mut scroll, node)) = content_q.get_mut(child) else {
                 continue;
             };
@@ -186,7 +159,7 @@ fn input_touch_pressed_move(
             if interaction != Interaction::Pressed {
                 continue;
             }
-            for child in children.iter() {
+            for &child in children.iter() {
                 let Ok(mut scroll) = content_q.get_mut(child) else {
                     continue;
                 };
@@ -217,7 +190,7 @@ fn scroll_events(
             #[cfg(feature = "extra_logs")]
             info!("Scroolling by {:#?}: {} movement", ev.unit, y);
 
-            for child in children.iter() {
+            for &child in children.iter() {
                 let Ok(mut scroll) = content_q.get_mut(child) else {
                     continue;
                 };