-
Notifications
You must be signed in to change notification settings - Fork 8.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
[Security Solution] New navigation panel bottom padding fix #132951
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
const wrapper = mount(Component); | ||
await waitFor(() => wrapper.find('[data-test-subj="queryInput"]').exists()); // check for presence of query input | ||
return wrapper; |
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.
unrelated, just fixing a flaky test that was failing randomly
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.
LGTM!
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
* fix panel nav bottom padding when index does not exist * fix unrelated flacky test * [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix' Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 159fe79)
💔 Build FailedFailed CI StepsMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @semd |
…#133006) * fix panel nav bottom padding when index does not exist * fix unrelated flacky test * [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix' Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 159fe79) Co-authored-by: Sergi Massaneda <sergi.massaneda@elastic.co> Co-authored-by: Spencer <spencer@elastic.co>
Summary
issue: #130023
The bug appears when the instance is new and the timeline indices do not yet exist.
before:
after:
The padding was added to the navigation panel based on the location path flag defined in the static config, but the timeline bottom bar is also hidden if the index does not exist, which was not checked by the
showTimeline
hook.I just moved the logic of checking the dataView values from the
bottom_bar
to theshowTimeline
hook, so the navigation will also consider this case when adding the padding.Checklist