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

Test PR for re-enabling vertical bar chart tests #47122

Conversation

dmlemeshko
Copy link
Member

@dmlemeshko dmlemeshko commented Oct 2, 2019

Testing #47106 to make sure it is not flaky

markov00 and others added 2 commits October 2, 2019 12:27
The current angular implementation of the tooltip, applied to the Apply button of the visualize editor, suffer from missing the mouseleave event in some cases. This cause the tooltip to remain open in some cases halting the test from clicking the underlying button.
The tooltip implementation will be soon replaced by EUI, so on this PR we are only unskipping the test and scrolling the visualize editor sidebar to avoid this overlap.
@dmlemeshko dmlemeshko added the test-matrix Use this label to ensure PRs are tested with matrix jobs label Oct 2, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@dmlemeshko dmlemeshko closed this Oct 2, 2019
@dmlemeshko dmlemeshko deleted the fix_unskipped_vb_multiple_split_test branch January 31, 2022 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test-matrix Use this label to ensure PRs are tested with matrix jobs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants