-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(accordion): only target direct child p tags #5291
fix(accordion): only target direct child p tags #5291
Conversation
Deploy preview for carbon-components-react failed. Built with commit a0e815a https://app.netlify.com/sites/carbon-components-react/deploys/5e3c6f4a02a24f000cf62a99 |
Deploy preview for carbon-elements failed. Built with commit a0e815a https://app.netlify.com/sites/carbon-elements/deploys/5e3c6f4af5b343000879aa39 |
Deploy preview for carbon-components-react failed. Built with commit ff3db3a https://app.netlify.com/sites/carbon-components-react/deploys/5e3d9bafcc82370008d6b7da |
Deploy preview for carbon-elements ready! Built with commit ff3db3a |
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 @tw15egan!
Closes #5285
We were incorrectly applying
body-long-01
text styles to ALLp
elements inside ofbx--accordion__content
. This makes sure we only target direct childp
tagsChangelog
Changed
p
tags instead of allp
tags inside ofbx--accordion__content
Testing / Reviewing
Add an inline-notification to an accordion and ensure the text is not altered.