Skip to content

Reset wrapping in label tooltip - #7535

Merged
lucasmerlin merged 2 commits into
mainfrom
lucas/fix-label-tooltip-wrapping
Sep 11, 2025
Merged

lucasmerlin merged 2 commits into
mainfrom
lucas/fix-label-tooltip-wrapping

Conversation

@lucasmerlin

Copy link
Copy Markdown
Collaborator

That PR changed the tooltip to preserve the wrapping, which made the tooltip kind of useless. With this PR the wrapping is reset for the tooltip.

@github-actions

Copy link
Copy Markdown

Preview available at https://egui-pr-preview.github.io/pr/7535-lucasfix-label-tooltip-wrapping
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin lucasmerlin added bug Something is broken egui labels Sep 11, 2025

@emilk emilk left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

let job = crate::text::LayoutJob {
sections: galley.job.sections.clone(),
text: galley.job.text.clone(),
..crate::text::LayoutJob::default()

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call

@lucasmerlin
lucasmerlin merged commit 2d3ecd3 into main Sep 11, 2025
49 checks passed
@lucasmerlin
lucasmerlin deleted the lucas/fix-label-tooltip-wrapping branch September 11, 2025 15:18
lucasmerlin added a commit that referenced this pull request Sep 11, 2025
* follow up to #7514

That PR changed the tooltip to preserve the wrapping, which made the
tooltip kind of useless. With this PR the wrapping is reset for the
tooltip.
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
* follow up to emilk#7514 

That PR changed the tooltip to preserve the wrapping, which made the
tooltip kind of useless. With this PR the wrapping is reset for the
tooltip.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is broken egui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants