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

[Code] fix folder collapse #35458

Merged
merged 1 commit into from
Apr 28, 2019
Merged

[Code] fix folder collapse #35458

merged 1 commit into from
Apr 28, 2019

Conversation

WangQianliang
Copy link
Contributor

Summary

Kapture 2019-04-23 at 18 04 04

@WangQianliang
Copy link
Contributor Author

Should we add a functional test on this? @mw-ding

@elasticmachine
Copy link
Contributor

💔 Build Failed

@mw-ding
Copy link
Contributor

mw-ding commented Apr 24, 2019

yes, let's add a test for this. also, can you change the base branch to master?

@mw-ding mw-ding self-requested a review April 24, 2019 03:22
@mw-ding
Copy link
Contributor

mw-ding commented Apr 24, 2019

or you can add a test case in this PR: #35216 since it has not been merged yet.

@WangQianliang WangQianliang marked this pull request as ready for review April 24, 2019 10:12
@WangQianliang WangQianliang changed the base branch from feature/merge-code to master April 24, 2019 10:12
@elasticmachine
Copy link
Contributor

💔 Build Failed

@WangQianliang
Copy link
Contributor Author

WangQianliang commented Apr 26, 2019

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@WangQianliang WangQianliang merged commit 7f1bf93 into elastic:master Apr 28, 2019
@WangQianliang WangQianliang deleted the code-fix-folder-collapse branch April 28, 2019 07:45
lizozom pushed a commit to lizozom/kibana that referenced this pull request Apr 29, 2019
zfy0701 added a commit that referenced this pull request Apr 30, 2019
#35819)

* test(code/frontend): more history functional test cases (#35216)

* fix(code/frontend): should not collapse folder after click a breadcrumb (#35458)

* [Code] fix typeahead highlight file (#35585)

* fix(code/frontend): should not collapse folder after click a breadcrumb

* fix(code/frontend): should highlight file when jump from typeahead

* separate to a independent test case

* fix(code/frontend): show loading in directory view (#35720)

* fix(code/frontend): fix highlight symbol (#35531)

* feature(code/frontend): retry loading symbols if language server is initializing (#35715)
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.

3 participants