Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

TextUiReader requires a parent Text to function, despite TextSpan docs #16794

Open
voximity opened this issue Dec 13, 2024 · 1 comment · May be fixed by #17415
Open

TextUiReader requires a parent Text to function, despite TextSpan docs #16794

voximity opened this issue Dec 13, 2024 · 1 comment · May be fixed by #17415
Labels
A-Text Rendering and layout for characters A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior S-Needs-Design This issue requires design work to think about how it would best be accomplished

Comments

@voximity
Copy link

Bevy version

Bevy 0.15.0

What you did

I constructed a root TextLayout with child TextSpans, as per the TextSpan documentation.

What went wrong

TextUiReader should allow me to properly iterate over the child TextSpans of the entity with TextLayout, but it will return an empty iterator if there is no root parent Text component.

I'm not sure what the expected behavior is here; if I don't want to use a root Text component, should TextUiReader still work? Or is the example for TextSpan incorrect?
@voximity voximity added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Dec 13, 2024
@BenjaminBrienen BenjaminBrienen added A-UI Graphical user interfaces, styles, layouts, and widgets S-Needs-Design This issue requires design work to think about how it would best be accomplished A-Text Rendering and layout for characters and removed S-Needs-Triage This issue needs to be labelled labels Dec 27, 2024
@BenjaminBrienen
Copy link
Contributor

When the expected behavior is made clear, I'll update the status again.

@rparrett rparrett linked a pull request Jan 17, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Text Rendering and layout for characters A-UI Graphical user interfaces, styles, layouts, and widgets C-Bug An unexpected or incorrect behavior S-Needs-Design This issue requires design work to think about how it would best be accomplished
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve TextSpan docs rparrett/bevy
2 participants
@voximity @BenjaminBrienen