Skip to content

multiple_text_input example layout fixes#24294

Open
ickshonpe wants to merge 2 commits into
bevyengine:mainfrom
ickshonpe:multiple-text-inputs-example-layout-fix
Open

multiple_text_input example layout fixes#24294
ickshonpe wants to merge 2 commits into
bevyengine:mainfrom
ickshonpe:multiple-text-inputs-example-layout-fix

Conversation

@ickshonpe
Copy link
Copy Markdown
Contributor

@ickshonpe ickshonpe commented May 14, 2026 •
edited
Loading

Objective

These are meant to be single line inputs, but:

image

A mess!

Solution

  1. Add TextLayout::no_wrap() to each input and their outputs.
  2. Add an intermediate node wrapping each output node applying clipping to hide the overflow.
fixed-inputs

Testing

cargo run --example multiple_text_inputs  --features="system_clipboard"

@ickshonpe
Fix the single line text inputs, so they actually are single line.
6d35e2b 
1. Add `TextLayout::no_wrap()` to each input and their outputs.
2. Clip the overflow of the output nodes by adding an intermediate wrapping node to apply the clipping.
@ickshonpe ickshonpe added C-Bug An unexpected or incorrect behavior A-UI Graphical user interfaces, styles, layouts, and widgets A-Text Rendering and layout for characters D-Straightforward Simple bug fixes and API improvements, docs, test and examples labels May 14, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in UI May 14, 2026
@ickshonpe ickshonpe added S-Needs-Review Needs reviewer attention (from anyone!) to move forward C-Examples An addition or correction to our examples labels May 14, 2026
@ickshonpe ickshonpe added this to the 0.19 milestone May 14, 2026
amtep
amtep approved these changes May 14, 2026
View reviewed changes
Comment thread examples/ui/text/multiple_text_inputs.rs Outdated
Comment thread examples/ui/text/multiple_text_inputs.rs Outdated
@ickshonpe ickshonpe changed the title mulitline_text_input example layout fixes multiline_text_input example layout fixes May 14, 2026
@ickshonpe ickshonpe changed the title multiline_text_input example layout fixes multiple_text_input example layout fixes May 14, 2026
@ickshonpe ickshonpe mentioned this pull request May 14, 2026
Closed
@JMS55 JMS55 added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@amtep amtep amtep approved these changes

@JMS55 JMS55 JMS55 approved these changes

Reviewers whose approvals may not affect merge requirements

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 C-Examples An addition or correction to our examples D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

UI
Status: Needs SME Triage

Milestone

0.19

Development

Successfully merging this pull request may close these issues.

3 participants

@ickshonpe @amtep @JMS55