From dfd3d61aae6a0dfe6e780f0571af73742863fa30 Mon Sep 17 00:00:00 2001 From: ickshonpe Date: Sun, 27 Oct 2024 22:39:32 +0000 Subject: [PATCH] Move `ContentSize` requirements from `Node` to the widget defining components (#16083) Missed this in the required components PR review. `ContentSize` isn't used by regular UI nodes, only those with intrinsically sized content that needs a measure func. Remove `ContentSize` from `Node`'s required components and add it to the required components of `Text` and `UiImage`. --------- Co-authored-by: Alice Cecile --- crates/bevy_ui/src/ui_node.rs | 3 +-- crates/bevy_ui/src/widget/text.rs | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/crates/bevy_ui/src/ui_node.rs b/crates/bevy_ui/src/ui_node.rs index 88b950b7b479b..46e25f975c132 100644 --- a/crates/bevy_ui/src/ui_node.rs +++ b/crates/bevy_ui/src/ui_node.rs @@ -1,4 +1,4 @@ -use crate::{widget::UiImageSize, ContentSize, FocusPolicy, UiRect, Val}; +use crate::{widget::UiImageSize, FocusPolicy, UiRect, Val}; use bevy_asset::Handle; use bevy_color::Color; use bevy_ecs::{prelude::*, system::SystemParam}; @@ -288,7 +288,6 @@ impl From<&Vec2> for ScrollPosition { BackgroundColor, BorderColor, BorderRadius, - ContentSize, FocusPolicy, ScrollPosition, Transform, diff --git a/crates/bevy_ui/src/widget/text.rs b/crates/bevy_ui/src/widget/text.rs index 90222ef5cd054..30817e05da835 100644 --- a/crates/bevy_ui/src/widget/text.rs +++ b/crates/bevy_ui/src/widget/text.rs @@ -102,7 +102,7 @@ pub struct TextBundle {} /// ``` #[derive(Component, Debug, Default, Clone, Deref, DerefMut, Reflect)] #[reflect(Component, Default, Debug)] -#[require(Node, TextLayout, TextFont, TextColor, TextNodeFlags)] +#[require(Node, TextLayout, TextFont, TextColor, TextNodeFlags, ContentSize)] pub struct Text(pub String); impl Text {