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(DataList): expose grid modifiers as properties #4969
Conversation
PF4 preview: https://patternfly-react-pr-4969.surge.sh |
Codecov Report
@@ Coverage Diff @@
## master #4969 +/- ##
=======================================
Coverage 52.15% 52.15%
=======================================
Files 530 530
Lines 9769 9770 +1
Branches 3597 3597
=======================================
+ Hits 5095 5096 +1
Misses 4013 4013
Partials 661 661
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
import { PickOptional } from '../../helpers/typeUtils'; | ||
|
||
export enum DataListGridBreakpoint { | ||
gridNone = 'grid-none', |
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 need to expose grid as well here
#4965 (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! Thanks @KKoukiou 🥳
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 please also update the demo app and integration test to check all breakpoints. We have had cases in the past where the breakpoints were not being applied as expected.
import { PickOptional } from '../../helpers/typeUtils'; | ||
|
||
export enum DataListGridBreakpoint { |
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.
You don't have to export an enum (strings work just fine), but if you do, make sure to add it to the gridBreakPoint
type in DataListProps
.
bc7e7e5
to
25c100d
Compare
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!! One small nit left
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, thanks @KKoukiou !
Your changes have been released in:
Thanks for your contribution! 🎉 |
Closes #4965