Move Wrap back to a subcomponent

This reverts commit 8824e3a.

Author: alice-i-cecile

crates/bevy_ui/src/entity.rs

@@ -3,7 +3,7 @@
 use crate::{
     layout_components::{
         flex::FlexLayout, LayoutStrategy, Offset, Overflow, PositionType, SizeConstraints, Spacing,
-        TextDirection, Wrap,
+        TextDirection,
     },
     widget::{Button, ImageMode},
     CalculatedSize, FocusPolicy, Interaction, Node, UiColor, UiImage,
@@ -39,8 +39,6 @@ pub struct NodeBundle {
     pub flex_layout: FlexLayout,
     /// The direction of the text
     pub text_direction: TextDirection,
-    /// Controls how the content wraps
-    pub wrap: Wrap,
     /// The behavior in case the node overflows its allocated space
     pub overflow: Overflow,
     /// Describes the color of the node
@@ -78,8 +76,6 @@ pub struct ImageBundle {
     pub flex_layout: FlexLayout,
     /// The direction of the text
     pub text_direction: TextDirection,
-    /// Controls how the content wraps
-    pub wrap: Wrap,
     /// The behavior in case the node overflows its allocated space
     pub overflow: Overflow,
     /// Configures how the image should scale
@@ -121,8 +117,6 @@ pub struct TextBundle {
     pub flex_layout: FlexLayout,
     /// The direction of the text
     pub text_direction: TextDirection,
-    /// Controls how the content wraps
-    pub wrap: Wrap,
     /// The behavior in case the node overflows its allocated space
     pub overflow: Overflow,
     /// Contains the text of the node
@@ -187,7 +181,6 @@ impl Default for TextBundle {
             size_constraints: Default::default(),
             flex_layout: Default::default(),
             text_direction: Default::default(),
-            wrap: Default::default(),
             overflow: Default::default(),
             transform: Default::default(),
             global_transform: Default::default(),
@@ -220,8 +213,6 @@ pub struct ButtonBundle {
     pub flex_layout: FlexLayout,
     /// The direction of the text
     pub text_direction: TextDirection,
-    /// Controls how the content wraps
-    pub wrap: Wrap,
     /// The behavior in case the node overflows its allocated space
     pub overflow: Overflow,
     /// Describes whether and how the button has been interacted with by the input

crates/bevy_ui/src/flex/convert.rs

@@ -1,8 +1,9 @@
 use crate::layout_components::{
     flex::{
         AlignContent, AlignItems, AlignSelf, FlexDirection, FlexLayoutQueryItem, JustifyContent,
+        Wrap,
     },
-    LayoutStrategy, PositionType, Wrap,
+    LayoutStrategy, PositionType,
 };
 use crate::{Size, UiRect, Val};
 
@@ -41,7 +42,7 @@ pub fn from_flex_layout(scale_factor: f64, value: FlexLayoutQueryItem<'_>) -> ta
         display: (*value.layout_strategy).into(),
         position_type: (*value.position_type).into(),
         flex_direction: value.flex_layout.flex_direction.into(),
-        flex_wrap: (*value.wrap).into(),
+        flex_wrap: value.flex_layout.wrap.into(),
         align_items: value.flex_layout.align_items.into(),
         align_self: value.flex_layout.align_self.into(),
         align_content: value.flex_layout.align_content.into(),

crates/bevy_ui/src/layout_components.rs

@@ -204,32 +204,12 @@ pub enum Overflow {
     Hidden,
 }
 
-/// Defines if child UI items appear on a single line or on multiple lines
-#[derive(
-    Component, Copy, Clone, PartialEq, Eq, Debug, Default, Serialize, Deserialize, Reflect,
-)]
-#[reflect_value(PartialEq, Serialize, Deserialize)]
-pub enum Wrap {
-    /// Single line, will overflow if needed
-    #[default]
-    NoWrap,
-    /// Multiple lines, if needed
-    Wrap,
-    /// Same as [`FlexWrap::Wrap`] but new lines will appear before the previous one
-    WrapReverse,
-}
-
 /// Flexbox-specific layout components
 pub mod flex {
-    use super::{
-        LayoutStrategy, Offset, Overflow, PositionType, SizeConstraints, Spacing, TextDirection,
-        Wrap,
-    };
-    use crate::Val;
-    use bevy_ecs::prelude::Component;
+
     use bevy_ecs::query::{Changed, Or, WorldQuery};
-    use bevy_reflect::prelude::*;
-    use serde::{Deserialize, Serialize};
+
+    use super::*;
 
     /// A query for all of the components need for flexbox layout.
     ///
@@ -250,8 +230,6 @@ pub mod flex {
         pub flex_layout: &'static FlexLayout,
         /// The direction of the text
         pub text_direction: &'static TextDirection,
-        /// Controls how the content wraps
-        pub wrap: &'static Wrap,
         /// The behavior in case the node overflows its allocated space
         pub overflow: &'static Overflow,
     }
@@ -286,6 +264,8 @@ pub mod flex {
         pub align_content: AlignContent,
         /// Aligns this containers items along the main-axis
         pub justify_content: JustifyContent,
+        /// Controls how the content wraps
+        pub wrap: Wrap,
         /// Defines how much a flexbox item should grow if there's space available
         pub grow: f32,
         /// How to shrink if there's not enough space available
@@ -302,6 +282,7 @@ pub mod flex {
                 align_self: Default::default(),
                 align_content: Default::default(),
                 justify_content: Default::default(),
+                wrap: Default::default(),
                 grow: 0.0,
                 shrink: 1.0,
                 basis: Val::Auto,
@@ -401,4 +382,17 @@ pub mod flex {
         /// Like [`JustifyContent::SpaceAround`] but with even spacing between items
         SpaceEvenly,
     }
+
+    /// Defines if flexbox items appear on a single line or on multiple lines
+    #[derive(Copy, Clone, PartialEq, Eq, Debug, Default, Serialize, Deserialize, Reflect)]
+    #[reflect_value(PartialEq, Serialize, Deserialize)]
+    pub enum Wrap {
+        /// Single line, will overflow if needed
+        #[default]
+        NoWrap,
+        /// Multiple lines, if needed
+        Wrap,
+        /// Same as [`FlexWrap::Wrap`] but new lines will appear before the previous one
+        WrapReverse,
+    }
 }