-
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
create common watch actions component #36056
create common watch actions component #36056
Conversation
Pinging @elastic/es-ui |
💚 Build Succeeded |
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.
Great work! Code LGTM, didn't test locally.
case ACTION_STATES.THROTTLED: | ||
return <EuiIcon type="clock" color="warning" />; | ||
case WATCH_STATES.DISABLED: | ||
return <EuiIcon type="minusInCircleFilled" color="#69707D" />; // EUI color: euiColorDarkShade |
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.
Was "subdued" too hard to see? If "subdued" doesn't work, then could you submit an issue in the EUI repo asking for this to be supported via a color name like "disabled" or something?
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.
subdued
works, thanks! i didn't see that as an option initially.
💚 Build Succeeded |
This PR addresses the following:
Simulation results
flyout underneath the titleindex.ts
file for common componentsScreenshots
Possible states:
Simulation results flyout: