Skip to content

Reset wrapping in label tooltip#7535

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

Reset wrapping in label tooltip#7535
lucasmerlin merged 2 commits intomainfrom
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
Copy link
Copy Markdown
Owner

@emilk emilk left a comment

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