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

CHE-3578: set focus on terminal after resizing #3654

Merged
merged 5 commits into from
Jan 11, 2017
Merged

CHE-3578: set focus on terminal after resizing #3654

merged 5 commits into from
Jan 11, 2017

Conversation

svor
Copy link
Contributor

@svor svor commented Jan 10, 2017

What does this PR do?

Sets focus on the terminal after resizing.

What issues does this PR fix or reference?

#3578

Previous behavior

Focus is lost when the terminal was minimized or maximized

New behavior

Focus sets when size of the terminal is changed

@svor svor self-assigned this Jan 10, 2017
@svor svor requested a review from vparfonov January 10, 2017 13:55
@vzhukovs
Copy link
Contributor

@vparfonov I suppose that this PR will break changes made by this previous commit 3da5a96#diff-e15936ac6286c2e6559b1194493b5b30L245

Because this method is called when terminal first time appears on the UI (when IDE starts).

@codenvy-ci
Copy link

@codenvy-ci
Copy link

@svor svor merged commit 1f52e25 into master Jan 11, 2017
@svor svor deleted the CHE-3578 branch January 11, 2017 13:54
@codenvy-ci
Copy link

@slemeur slemeur added this to the 5.1.0 milestone Jan 18, 2017
@slemeur slemeur added the kind/bug Outline of a bug - must adhere to the bug report template. label Jan 18, 2017
@JamesDrummond JamesDrummond mentioned this pull request Jan 31, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants