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.

Sign up for GitHub

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

Jump to bottom

Fix prose-invert when used with colors in light mode #315

Merged
merged 2 commits into from
Aug 30, 2023
Merged

Fix prose-invert when used with colors in light mode #315

merged 2 commits into from
Aug 30, 2023

Conversation

reinink
Copy link
Member

@reinink reinink commented Aug 30, 2023 •
edited
Loading

Fixes #228

This PR fixes an issue where prose-invert wasn't working when used with a prose-{color} modifier while in light mode. For example:

<div class="prose prose-invert prose-slate bg-slate-900 p-12">
  Lorem ipsum dolor sit amet...
</div>

Prior to this fix you'd get this:

image

With this change, you get this:

image

The solution here is what @adamwathan suggested in the linked issue, where we need to make sure that the invert styles come last in the list of modifiers — which causes them to be generated last in the outputted CSS.

@vercel
Copy link

vercel bot commented Aug 30, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
tailwindcss-typography ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 30, 2023 7:43pm

@thecrypticace
Copy link
Contributor

LGTM

@reinink reinink merged commit 0339c42 into master Aug 30, 2023
4 checks passed
@reinink reinink deleted the fix-invert-with-colors branch August 30, 2023 20:09
@reinink reinink mentioned this pull request Aug 30, 2023
Closed
This was referenced May 21, 2024
Merged
Merged
@Leeze555 Leeze555 mentioned this pull request May 31, 2024
Open
This was referenced Jun 12, 2024
Merged
Merged
@b-y-f b-y-f mentioned this pull request Jul 4, 2024
Open
@Juneid19 Juneid19 mentioned this pull request Jul 5, 2024
Open
@b-y-f b-y-f mentioned this pull request Jul 6, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 21, 2024
Open
@jones58 jones58 mentioned this pull request Jul 27, 2024
Merged
This was referenced Jul 27, 2024
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
This was referenced Oct 6, 2024
Open
Open
Open
Open
Open
Open
Open
Open
Open
This was referenced Oct 15, 2024
Open
Open
Open
Open
Open
Open
This was referenced Oct 22, 2024
Open
Open
Open
Open
Open
@Vishesh7797 Vishesh7797 mentioned this pull request Oct 26, 2024
Open
This was referenced Oct 27, 2024
Open
Open
Open
Open
Open
This was referenced Nov 5, 2024
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

prose-invert not working as expected
2 participants
@reinink @thecrypticace