-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Polish/scraper overlay #12317
Polish/scraper overlay #12317
Conversation
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
…ets exist Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
Co-Authored-By: Iris Scholten <ischolten.is@gmail.com>
} = this.props | ||
|
||
return ( | ||
<OverlayContainer maxWidth={600}> |
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.
Nice!!
icon?: IconFont | ||
} | ||
|
||
const Alert: SFC<Props> = ({children, color, icon}) => { |
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.
Yay an alert component! :)
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.
Looks good :)
Closes #12310
Change the design of the create scraper overlay to be more consistent with other overlays
Remove UI elements from when the form was inside onboarding
Construct a scraper object in overlay local state instead of using redux to store
Delete vestigial scraper creation components
Introduce clockface
Alert
componentShow alert in Telegrafs and Scrapers page when user has no buckets
Disabled
Create
button in Telegrafs and Scrapers page when user has no bucketsUpdate E2E tests
Checklist