fix: prevent name collisions

evidence-dev / evidence

Business intelligence as code: build fast, interactive data visualizations in pure SQL and markdown

https://evidence.dev

MIT License

3.38k stars 164 forks source link

fix: prevent name collisions #1912

Closed ItsMeBrianD closed 6 days ago

ItsMeBrianD commented 2 weeks ago

Description

Before the introduction of tailwind, we were using the container class name in a few places, once tailwind was introduced there were collisions that broke a few niche scenarios.

This renames all of the relevant classes to prevent the collision

Checklist

~~- [ ] For UI or styling changes, I have added a screenshot or gif showing before & after~~

[x] I have added a changeset ~~- [ ] I have added to the docs where applicable~~ ~~- [ ] I have added to the VS Code extension where applicable~~

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: c819f86cd29e4faf1747f0c14a0a3803ccf365fe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages

| Name | Type | | ----------------------------- | ----- | | @evidence-dev/core-components | Patch | | @evidence-dev/evidence | Patch | | my-evidence-project | Patch | | @evidence-dev/components | Patch | | evidence-test-environment | Patch |

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

netlify[bot] commented 2 weeks ago

Deploy Preview for next-docs-evidence ready!

Name	Link
Latest commit	c819f86cd29e4faf1747f0c14a0a3803ccf365fe
Latest deploy log	https://app.netlify.com/sites/next-docs-evidence/deploys/66330ab577af420008972d73
Deploy Preview	https://deploy-preview-1912--next-docs-evidence.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.