Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix focusable parameter for Window and Dialog #225

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

igordmn
Copy link
Collaborator

@igordmn igordmn commented May 30, 2022

setFocusable makes the root component focusable, not the window itself. For Window we should use setFocusableWindowState

setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
@igordmn igordmn requested a review from dima-avdeev-jb May 30, 2022 13:32
@igordmn igordmn merged commit 6fe4d1d into jb-main Jun 2, 2022
@igordmn igordmn deleted the feature/fixfocusablewindow branch June 2, 2022 15:44
eymar pushed a commit that referenced this pull request Jun 27, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
eymar pushed a commit that referenced this pull request Jun 28, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
igordmn added a commit that referenced this pull request Aug 18, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
igordmn added a commit that referenced this pull request Aug 18, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
eymar pushed a commit that referenced this pull request Oct 26, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
eymar pushed a commit that referenced this pull request Nov 16, 2022
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
eymar pushed a commit that referenced this pull request Jan 13, 2023
setFocusable make the root component focusable, not the window itself. For Window we should use setFocusableWindowState
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