-
Notifications
You must be signed in to change notification settings - Fork 359
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(Popper): updated default appendTo prop for Penta styling #10675
Conversation
Preview: https://patternfly-react-pr-10675.surge.sh A11y report: https://patternfly-react-pr-10675-a11y.surge.sh |
Are there any docs - in accessibilty docs or in component docs that advise against appending to document body that we will want to update as a result of this change? |
4fbd4f9
to
02aa4f2
Compare
@nicolethoen not that I could find when searching document body or append in the org repo. I did update Dropdown and Select a teeny bit with the latest update, though. |
@@ -27,7 +27,6 @@ DualListSelectorListItem as DLSListItemDeprecated, | |||
DualListSelectorControlsWrapper as DLSControlsWrapperDeprecated, | |||
DualListSelectorControl as DLSControlDeprecated, | |||
DualListSelectorTree as DLSTreeDeprecated, | |||
DualListSelectorTreeItemData as DLSTreeItemDataDeprecated, |
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.
Note that this is unrelated to the PR itself, but should help remove the warning when building locally.
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.
L🍕TM!
Your changes have been released in:
Thanks for your contribution! 🎉 |
What: Closes #10671
Additional issues: