-
Notifications
You must be signed in to change notification settings - Fork 351
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
feat(EmptyState): Convert empty state to TypeScript #1917
Conversation
PatternFly-React preview: https://1917-pr-patternfly-react-patternfly.surge.sh |
Codecov Report
@@ Coverage Diff @@
## master #1917 +/- ##
==========================================
- Coverage 81.84% 81.84% -0.01%
==========================================
Files 628 628
Lines 7112 7139 +27
Branches 236 243 +7
==========================================
+ Hits 5821 5843 +22
Misses 1163 1163
- Partials 128 133 +5
Continue to review full report at Codecov.
|
@rebeccaalpert can you also add integration tests for this component as outlined on this README? |
packages/patternfly-4/react-core/src/components/EmptyState/EmptyState.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyState.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateBody.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateBody.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateIcon.tsx
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateIcon.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateSecondaryActions.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyState.tsx
Outdated
Show resolved
Hide resolved
packages/patternfly-4/react-core/src/components/EmptyState/EmptyStateIcon.tsx
Outdated
Show resolved
Hide resolved
9abedff
to
b952967
Compare
b952967
to
15e9fab
Compare
Added test/demo and addressed PR feedback. |
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 great. Thanks!
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
} | ||
} | ||
|
||
export default EmptyStateDemo; |
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.
Remove export default
Refactored empty state to TypeScript.
Fixes #1991.