-
Notifications
You must be signed in to change notification settings - Fork 135
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
Update Search empty state #2975
base: master
Are you sure you want to change the base?
Conversation
EmptyStateHeader, | ||
EmptyStateIcon, | ||
} from '@patternfly/react-core/dist/dynamic/components/EmptyState'; | ||
// import { Button } from '@patternfly/react-core/dist/dynamic/components/Button'; |
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.
Can you remove the commented code?
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.
@Hyperkid123 We have two buttons that need to be activated in follow-up PRs/Jira and all the styling has been done here. Can we leave it all in? Otherwise, there's not much point to this PR cc @fhlavac
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.
We need https://issues.redhat.com/browse/RHCLOUD-36235 to activate the Launch button. @Hyperkid123 can you please take a look if anything needs to be added to the ticket? If it is good, I could do it to unblock this one (I know it's Chrome UI, so if you or the Services team have capacity, we can also assign it to someone else).
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.
I don't see any reason why you could not work on it.
</TextContent> | ||
</EmptyStateBody> | ||
{/* <EmptyStateFooter className="pf-v5-u-mt-md"> |
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.
^^
Jira: https://issues.redhat.com/browse/RHCLOUD-36187
Old
New