Skip to content
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(modal): override base paragraph styles in modal content #8046

Merged

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Mar 9, 2021

Closes #8043

This PR overrides the system wide text style rules on paragraphs only within modals (similar to existing overrides on other components e.g. accordion)

Testing / Reviewing

Confirm the text styles are correct in our modal storybook examples

@emyarod emyarod force-pushed the 8043-modal-body-content-token branch from 147786a to 4711794 Compare March 9, 2021 17:51
@netlify
Copy link

netlify bot commented Mar 9, 2021

Deploy preview for carbon-elements ready!

Built with commit 147786a

https://deploy-preview-8046--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Mar 9, 2021

Deploy preview for carbon-components-react ready!

Built with commit 147786a

https://deploy-preview-8046--carbon-components-react.netlify.app

@netlify
Copy link

netlify bot commented Mar 9, 2021

Deploy preview for carbon-elements ready!

Built with commit ef39f58

https://deploy-preview-8046--carbon-elements.netlify.app

Copy link
Member

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 ✅

@netlify
Copy link

netlify bot commented Mar 9, 2021

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit ef39f58

https://deploy-preview-8046--carbon-components-react.netlify.app

Copy link
Member

@laurenmrice laurenmrice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@kodiakhq kodiakhq bot merged commit 7936d1f into carbon-design-system:main Mar 9, 2021
@emyarod emyarod deleted the 8043-modal-body-content-token branch March 10, 2021 20:16
@emyarod emyarod mentioned this pull request Mar 22, 2021
87 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[modal] incorrect token for body content
4 participants