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(Popper, misc): update reference to triggerRef, components to use popperRef/triggerRef #8733
Conversation
Preview: https://patternfly-react-pr-8733.surge.sh A11y report: https://patternfly-react-pr-8733-a11y.surge.sh |
5d4807d
to
04684df
Compare
Noticed a bit of an odd behavior while I was fixing an integration test where when the window is narrowed, there is a white space that is added to the right of the screen of the demo app. I've went back and tested this on a plain v5 branch and still see the issue, so I'm not sure what's going on. For now I've updated the test. |
packages/react-core/src/components/ClipboardCopy/ClipboardCopyButton.tsx
Show resolved
Hide resolved
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 opened a follow up codemod issue patternfly/pf-codemods#296
@kmcfaul i'd love if you double check that there are no other breaking changes in this PR we should note in the codemod issue.
Your changes have been released in:
Thanks for your contribution! 🎉 |
What: Closes #8481
reference
totriggerRef
.Popper
to passtriggerRef
andpopperRef
so the wrapping divs are removed.ref
toInputGroup
(forSearchInput
).trigger
withtriggerRef
to minimize the amount of structural changes.Tooltip
andPopover
requiretriggerRef
to remove the wrapping div on the trigger, other components should include the references automatically. Not all components have exposedref
props so they use internal refs to pass to Popper.