-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[ML] Fixing position of job management search bar error #30251
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
[ML] Fixing position of job management search bar error #30251
Conversation
|
Pinging @elastic/ml-ui |
alvarezmelissa87
left a comment
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 ⚡️
💚 Build Succeeded |
walterra
left a comment
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
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.
450bab1 to
255b887
Compare
peteharverson
left a comment
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.
Latest change LGTM
💚 Build Succeeded |
* [ML] Style tweaks for job management in K7 * adjusting search bar error * removing custom title style for EuiTitle
* [ML] Style tweaks for job management in K7 * adjusting search bar error * removing custom title style for EuiTitle
* [ML] Style tweaks for job management in K7 * adjusting search bar error * removing custom title style for EuiTitle

When entering invalid text into the search bar, then error is displayed in a callout which throws off the layout of the page, especially if the multi-select menu is visible.
This changes the error to be a normal eui form validation error.
Before:

After:

Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers