Skip to content

Don't focus Areas, Windows and ScrollAreas#7827

Merged
emilk merged 4 commits intomainfrom
lucas/non-focus-resize-handle
Jan 5, 2026
Merged

Don't focus Areas, Windows and ScrollAreas#7827
emilk merged 4 commits intomainfrom
lucas/non-focus-resize-handle

Conversation

@lucasmerlin
Copy link
Copy Markdown
Collaborator

Currently, tabbing through egui demo app, there are a lot of widgets that have invisible focus. Tabbing into a window for example takes 10 (!) presses of the tab key before the first widget within the window is focused. Before that, the focus moves to each resize handle, the scroll area and the scroll bar. At that point a user might think the focus is entirely broken.

This pr removes the focusable sense from all these elements. Anything that can be focused should somehow indicate that it currently has focus, or the user could get frustrated.

It also adds a debug flag to always show the focused widget, so it's easier to debug these cases

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 2, 2026

Preview available at https://egui-pr-preview.github.io/pr/7827-lucasnon-focus-resize-handle
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

@emilk emilk added bug Something is broken egui labels Jan 5, 2026
@emilk emilk merged commit f9bf0ee into main Jan 5, 2026
47 of 48 checks passed
@emilk emilk deleted the lucas/non-focus-resize-handle branch January 5, 2026 11:50
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
Currently, tabbing through egui demo app, there are a lot of widgets
that have invisible focus. Tabbing into a window for example takes 10
(!) presses of the tab key before the first widget within the window is
focused. Before that, the focus moves to each resize handle, the scroll
area and the scroll bar. At that point a user might think the focus is
entirely broken.

This pr removes the focusable sense from all these elements. Anything
that can be focused should somehow indicate that it currently has focus,
or the user could get frustrated.

It also adds a debug flag to always show the focused widget, so it's
easier to debug these cases
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