-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Unexpected loss of focus on the project #3170
Conversation
@@ -120,20 +106,6 @@ public void onWsAgentStarted(WsAgentStateEvent event) { | |||
restoreTerminal(); | |||
|
|||
machinePortProvider.get(); | |||
/* Do not show terminal on factories by default */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why we remove this check here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As terminal doesn't get focus at startup we can show terminal after consuming factory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's leave part closed for factories
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1119/ |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1125/ |
* Add ability to set up manual focus to the terminal panel * Fix imports * Add ability to auto select first node when tree becomes visible and it has at least one node * Fixes * Fixes
Add ability to setup focus on Project Explorer part when IDE has already initialized.
Also was removed auto setting up focus on the terminal. So the terminal now handles
onblur
andonfocus
events to control own state.Related issue: #2795
@riuvshin @ashumilova @vparfonov review it, please.