Skip to content

Respect WidgetVisuals::expansion in TextEdit #8013

Merged
lucasmerlin merged 4 commits intomainfrom
lucas/fix-atom-text-edit-size
Mar 24, 2026
Merged

Respect WidgetVisuals::expansion in TextEdit #8013
lucasmerlin merged 4 commits intomainfrom
lucas/fix-atom-text-edit-size

Conversation

@lucasmerlin
Copy link
Copy Markdown
Collaborator

@lucasmerlin lucasmerlin commented Mar 24, 2026

This broke in #7587

The atom-based TextEdit conversion dropped the `visuals.expansion`
that was previously applied via `response.rect.expand(expansion)`.
Apply it the same way Button does: add expansion to inner_margin
and use negative outer_margin so the painted frame expands outward.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2026

Preview available at https://egui-pr-preview.github.io/pr/8013-lucasfix-atom-text-edit-size
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

View snapshot changes at kitdiff

@lucasmerlin lucasmerlin changed the title Respect WidgetVisuals::expansion in atom-based TextEdit frame Respect WidgetVisuals::expansion in TextEdit Mar 24, 2026
@lucasmerlin lucasmerlin merged commit 4feac89 into main Mar 24, 2026
46 checks passed
@lucasmerlin lucasmerlin deleted the lucas/fix-atom-text-edit-size branch March 24, 2026 16:33
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
This broke in emilk#7587

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants