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

Correct handle of input events in navigate to file dialogue #9369

Merged
merged 1 commit into from
Apr 11, 2018

Conversation

vzhukovs
Copy link
Contributor

What does this PR do?

This changes proposal provides changes to correct handling input events in Navigate to file dialogue. At first keyDown events where replaced by keyUp events, because it could cause incorrect construction of search request when user search a file with a single character.

Signed-off-by: Vladyslav Zhukovskyi vzhukovs@redhat.com

What issues does this PR fix or reference?

#8293

Release Notes

N/A

Docs PR

N/A

Signed-off-by: Vladyslav Zhukovskyi <vzhukovs@redhat.com>
@vzhukovs vzhukovs added kind/bug Outline of a bug - must adhere to the bug report template. team/ide severity/P1 Has a major impact to usage or development of the system. e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. labels Apr 10, 2018
@vzhukovs vzhukovs added this to the 6.4.0 milestone Apr 10, 2018
@vzhukovs vzhukovs self-assigned this Apr 10, 2018
@vzhukovs vzhukovs requested a review from vparfonov April 10, 2018 14:47
@vzhukovs
Copy link
Contributor Author

ci-test

@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Apr 10, 2018
@codenvy-ci
Copy link

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:9369
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@musienko-maxim
Copy link
Contributor

The selenium session did not find a new regression

@vzhukovs vzhukovs merged commit bff7925 into master Apr 11, 2018
@vzhukovs vzhukovs deleted the che#8293 branch April 11, 2018 11:54
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Apr 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
No open projects
IDE
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

5 participants