feat(misc): Replace Text, TextContent, TextList and TextListItem with Content - PART 1#10611
Conversation
|
Preview: https://patternfly-react-pr-10611.surge.sh A11y report: https://patternfly-react-pr-10611-a11y.surge.sh |
7922559 to
d49a920
Compare
d49a920 to
6aee8c8
Compare
kmcfaul
left a comment
There was a problem hiding this comment.
We should update Content.tsx to export as Content, it's still being exported as Text and is referred to as Text in the docs and examples (including the imports and usage in examples). The examples should use Content instead of the Text suite (maybe keeping a couple Text-using examples if we are keeping the Text components around, but my impression was they were being removed?).
Do we want to remove TextContent, TextList, TextListItem or are we keeping them for now? Not a blocker if we want to keep them.
|
@kmcfaul The reason for doing the changes separately in two PRs is:
|
|
Your changes have been released in:
Thanks for your contribution! 🎉 |
What: Closes #10395, #10390
This is a PART 1, it is not a breaking change yet, see this comment.
Part 2 PR: #10643
Codemod PR: patternfly/pf-codemods#675
Org issue to replace Text: patternfly/patternfly-org#4087
(link to previous closed PR: #10579)