Skip to content

Comments

Prevent focus interception by a browser when a backing html input is focused#2452

Merged
eymar merged 2 commits intojb-mainfrom
ok/fix_tab_focus_in_textfields
Oct 10, 2025
Merged

Prevent focus interception by a browser when a backing html input is focused#2452
eymar merged 2 commits intojb-mainfrom
ok/fix_tab_focus_in_textfields

Conversation

@eymar
Copy link
Member

@eymar eymar commented Oct 8, 2025

When handling Tab keydown event, we call preventDefault() to keep the focus in the Compose scene. Compose scene will process the keydown itself and move the focus within itself.

Fixes https://youtrack.jetbrains.com/issue/CMP-9041

Testing

  • Added a test
  • This should be tested by QA

Release Notes

Fixes - Web

  • Fix focus with Tab behaviour in Text Fields

@eymar eymar requested a review from Schahen October 8, 2025 09:53
Copy link

@Schahen Schahen left a comment

Choose a reason for hiding this comment

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

Thanks!

@eymar eymar merged commit e189e68 into jb-main Oct 10, 2025
10 checks passed
@eymar eymar deleted the ok/fix_tab_focus_in_textfields branch October 10, 2025 12:50
eymar added a commit that referenced this pull request Oct 22, 2025
…focused (#2452)

When handling Tab keydown event, we call `preventDefault()` to keep the
focus in the Compose scene. Compose scene will process the keydown
itself and move the focus within itself.

Fixes https://youtrack.jetbrains.com/issue/CMP-9041

## Testing
- Added a test
- This should be tested by QA

## Release Notes
### Fixes - Web
- Fix focus with Tab behaviour in Text Fields
sekater added a commit that referenced this pull request Oct 22, 2025
…html input is focused (#2452) (#2507)

When handling Tab keydown event, we call `preventDefault()` to keep the
focus in the Compose scene. Compose scene will process the keydown
itself and move the focus within itself.

Fixes https://youtrack.jetbrains.com/issue/CMP-9041

## Testing
- Added a test
- This should be tested by QA

## Release Notes
#2452
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.

2 participants