Skip to content

Simplify fn Events::handle_resize#685

Merged
dhardy merged 2 commits intomasterfrom
push-plvrmmyyvpuo
Mar 24, 2026
Merged

Simplify fn Events::handle_resize#685
dhardy merged 2 commits intomasterfrom
push-plvrmmyyvpuo

Conversation

@dhardy
Copy link
Copy Markdown
Collaborator

@dhardy dhardy commented Mar 24, 2026

Use a simpler API signature. Allow EditBox to assume its contents do not need size_rules or set_rect called again (they don't in this case, though usually calling ConfigCx::resize() indicates otherwise).

Alternative considered: make ActionResize an enum and all use-cases specify the type of resize required. But: it's considerable extra complexity and in almost all other cases the two variants would be equivalent. This might be worth revisiting if further changes to the layout-solving API take place.

@dhardy dhardy merged commit 63f1e79 into master Mar 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant