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(consent-manager)!: convert to CSS modules #305
Conversation
π¦ Changeset detectedLatest commit: f7eaa97 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This pull request is being automatically deployed with Vercel (learn more). π Inspect: https://vercel.com/hashicorp/react-components/Bi2yp5RkqphU5D8maZFEddnCjULt |
e8a5f1a
to
96f3b59
Compare
.changeset/rich-bugs-heal.md
Outdated
|
||
- π₯β¨ BREAKING CHANGE: Converts to CSS modules. | ||
- Consumers will need to remove any `@hashicorp/react-consent-manager/style.css` imports. | ||
- We've retained a `id="g-consent-manager` on the root element. |
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.
Added a debt task for this:
https://app.asana.com/0/1100423001970639/1200916857009754/f
@@ -80,7 +82,7 @@ export default class ConsentManager extends Component { | |||
|
|||
render() { | |||
return ( | |||
<div id="g-consent-manager"> | |||
<div className={classNames(s.root, this.props.className)}> |
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.
Added support for className
to ensure consumers can override hiding from Percy if needed.
@@ -12,7 +12,6 @@ | |||
@import '../packages/enterprise-alert/style.css'; | |||
@import '../packages/featured-slider/style.css'; | |||
@import '../packages/content-cta/style.css'; | |||
@import '../packages/consent-manager/style.css'; |
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.
πͺ π
@@ -0,0 +1,5 @@ | |||
.root { | |||
@media only percy { |
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.
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!
{/* TODO: this should most likely be a button (https://app.asana.com/0/1100423001970639/1199667739287943/f) */} | ||
{/* eslint-disable-next-line jsx-a11y/anchor-is-valid */} |
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.
nice cleanup!
ποΈ Asana Task
π Preview Link
This PR updates
@hashicorp/react-consent-manager
to use CSS modules.PR Checklist π
Items in this checklist may not may not apply to your PR, but please consider each item carefully.