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
fix(DataList, Table): update hoverable reference to clickable #9083
Conversation
Preview: https://patternfly-react-pr-9083.surge.sh A11y report: https://patternfly-react-pr-9083-a11y.surge.sh |
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, just needs a codemod for the prop rename.
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.
This looks good. One thing I noticed, which could be a followup since it's something that needs updating in other places, is in the DataListClickableRows example and the Tr component code, the usage of pf-screen-reader
just needs to be updated to pf-v5-screen-reader
(this one slipped by during the collision work)
I'll fix it |
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.
is there a code mod for this one?
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.
🚀
|
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.
The update to the hardcoded screen reader classes looks good as well 🎉
What: Closes #9001
This is clean up after core work: patternfly/patternfly#5397
basically, hoverable classnames for DataList and Table have been changed to clickable.
Changes here:
isHoverable
prop toisClickable