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] Full Screen of Rule Preview breaking the UI under Rule Creation form #146687
[Security Solution] Full Screen of Rule Preview breaking the UI under Rule Creation form #146687
Conversation
… Rule Creation form (elastic#145954)
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @e40pud |
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.
Verified issue beforehand, then checked out and tested fix and the Rule Preview table now goes into full screen mode without issue -- LGTM! 🎉
Note, in testing I found these two other UI issues. Not sure if they're currently tracked, but wanted to make note:
- Due to the smaller fixed width of the split pane, the
Inspect
hover button covers up theKeyboard Shortcuts
button, making it quite difficult to click:
- Re-sizing the window past a certain min-width results in the Rule Preview table disappearing until
Refresh
is clicked:
… Rule Creation form (elastic#146687) ## Summary These changes fix broken fullscreen table that displays the rule preview results. Fullscreen table before fix: <img width="1879" alt="Screenshot 2022-11-30 at 14 46 00" src="https://user-images.githubusercontent.com/2700761/204812399-52138490-a42a-4ea2-bd00-04680375580a.png"> Fullscreen table after fix: <img width="1881" alt="Screenshot 2022-11-30 at 14 45 20" src="https://user-images.githubusercontent.com/2700761/204812419-5b528ae5-02e4-4808-adcc-14ede7cec56b.png"> Ticket: elastic#145954 Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit d9b690f)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
… under Rule Creation form (#146687) (#146736) # Backport This will backport the following commits from `main` to `8.6`: - [[Security Solution] Full Screen of Rule Preview breaking the UI under Rule Creation form (#146687)](#146687) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"ievgen.sorokopud@elastic.co"},"sourceCommit":{"committedDate":"2022-11-30T17:57:30Z","message":"[Security Solution] Full Screen of Rule Preview breaking the UI under Rule Creation form (#146687)\n\n## Summary\r\n\r\nThese changes fix broken fullscreen table that displays the rule preview\r\nresults.\r\n\r\nFullscreen table before fix:\r\n\r\n<img width=\"1879\" alt=\"Screenshot 2022-11-30 at 14 46 00\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812399-52138490-a42a-4ea2-bd00-04680375580a.png\">\r\n\r\nFullscreen table after fix:\r\n\r\n<img width=\"1881\" alt=\"Screenshot 2022-11-30 at 14 45 20\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812419-5b528ae5-02e4-4808-adcc-14ede7cec56b.png\">\r\n\r\nTicket: #145954\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"d9b690f97a450fa29ea602e8181861045242c35f","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Detections and Resp","Team: SecuritySolution","Team:Detection Alerts","backport:prev-minor","ci:cloud-deploy","v8.7.0"],"number":146687,"url":"#146687 Solution] Full Screen of Rule Preview breaking the UI under Rule Creation form (#146687)\n\n## Summary\r\n\r\nThese changes fix broken fullscreen table that displays the rule preview\r\nresults.\r\n\r\nFullscreen table before fix:\r\n\r\n<img width=\"1879\" alt=\"Screenshot 2022-11-30 at 14 46 00\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812399-52138490-a42a-4ea2-bd00-04680375580a.png\">\r\n\r\nFullscreen table after fix:\r\n\r\n<img width=\"1881\" alt=\"Screenshot 2022-11-30 at 14 45 20\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812419-5b528ae5-02e4-4808-adcc-14ede7cec56b.png\">\r\n\r\nTicket: #145954\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"d9b690f97a450fa29ea602e8181861045242c35f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"#146687 Solution] Full Screen of Rule Preview breaking the UI under Rule Creation form (#146687)\n\n## Summary\r\n\r\nThese changes fix broken fullscreen table that displays the rule preview\r\nresults.\r\n\r\nFullscreen table before fix:\r\n\r\n<img width=\"1879\" alt=\"Screenshot 2022-11-30 at 14 46 00\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812399-52138490-a42a-4ea2-bd00-04680375580a.png\">\r\n\r\nFullscreen table after fix:\r\n\r\n<img width=\"1881\" alt=\"Screenshot 2022-11-30 at 14 45 20\"\r\nsrc=\"https://user-images.githubusercontent.com/2700761/204812419-5b528ae5-02e4-4808-adcc-14ede7cec56b.png\">\r\n\r\nTicket: #145954\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"d9b690f97a450fa29ea602e8181861045242c35f"}}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <ievgen.sorokopud@elastic.co>
Thanks @spong we've seen second issue, but not the first one. I will make a note of those and will address them in one of the upcoming releases. |
Summary
These changes fix broken fullscreen table that displays the rule preview results.
Fullscreen table before fix:
Fullscreen table after fix:
Ticket: #145954