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

Add nx- tailwind prefix #911

Merged
merged 4 commits into from Oct 20, 2022
Merged

Add nx- tailwind prefix #911

merged 4 commits into from Oct 20, 2022

Conversation

shuding
Copy link
Owner

@shuding shuding commented Oct 20, 2022

We need to isolate these classnames in case the website itself also uses Tailwind.

Seems like dark + prefix doesn't always work: dark:nx-shadow-[0_-12px_16px_#111] changing to darkMode: ['class', 'html[class~="dark"]'] fixes it, see tailwindlabs/tailwindcss#5754

@changeset-bot
Copy link

changeset-bot bot commented Oct 20, 2022

🦋 Changeset detected

Latest commit: 5ae1bcb

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

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

@vercel
Copy link

vercel bot commented Oct 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nextra-theme-docs-dev ✅ Ready (Inspect) Visit Preview Oct 20, 2022 at 10:58PM (UTC)
nextra-v2 ✅ Ready (Inspect) Visit Preview Oct 20, 2022 at 10:58PM (UTC)
1 Ignored Deployment
Name Status Preview Updated
nextra ⬜️ Ignored (Inspect) Oct 20, 2022 at 10:58PM (UTC)

@shuding
Copy link
Owner Author

shuding commented Oct 20, 2022

cc @B2o5T this is a pretty big refactoring and it will affect some other PRs 🙏 (however this should not break users' code unless they're doing something wrong)

@shuding shuding merged commit 580c433 into core Oct 20, 2022
@shuding shuding deleted the shu/46e6 branch October 20, 2022 22:58
tatukoivisto pushed a commit to tatukoivisto/nextra that referenced this pull request Aug 20, 2023
* add nx- tailwind prefix

* update

* fix dark mode

* add changeset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant