-
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] UI break for ML rule card under create a new rule for Basic License #145508
Comments
Pinging @elastic/security-solution (Team: SecuritySolution) |
@karanbirsingh-qasource Please review!! |
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Issue has been fixed in 8.7.0 by Upgrade EUI to v70.2.4. |
## Summary closes #145508 This workaround was added in 7832378. A non-temporary fix was pushed up in 8.7 (#144845), but rather than create a new EUI backport just for this issue, I'm opting to fix the 8.6-only workaround. ### Before <img src="https://user-images.githubusercontent.com/61860752/202394082-c82006af-1492-4537-a354-8e2f60b4b45e.png" alt=""> ### After <img width="909" alt="" src="https://user-images.githubusercontent.com/549407/203149130-a5cc0a69-d736-4e89-9d26-6e5e870238b4.png"> ### Checklist - [x] Any UI touched in this PR is usable by keyboard only (learn more about [keyboard accessibility](https://webaim.org/techniques/keyboard/)) - [x] This renders correctly on smaller devices using a responsive layout. (You can test this [in your browser](https://www.browserstack.com/guide/responsive-testing-on-local-server)) - [x] This was checked for [cross-browser compatibility](https://www.elastic.co/support/matrix#matrix_browsers)
Fixed in 8.6.0 in #145932 |
@deepikakeshav-qasource can you please validate the fix on next BC? Thanks :) |
Describe the bug
UI break for ML rule card under create a new rule for Basic License
Build info
Preconditions
Steps to Reproduce
Actual Result
UI break for ML rule card under create a new rule for Basic License
Expected Result
UI should not be break for ML rule card under create a new rule for Basic License
Screen-shot
The text was updated successfully, but these errors were encountered: