Skip to content
This repository was archived by the owner on Apr 6, 2021. It is now read-only.

Dark theme not compatible with prose / typography using new JIT engine #191

Closed
DoctorDerek opened this issue Mar 30, 2021 · 1 comment
Closed

Comments

@DoctorDerek
Copy link

DoctorDerek commented Mar 30, 2021

What version of @tailwindcss/jit are you using?

v0.1.7

What version of Node.js are you using?

v14.15.1

What build tool (or framework if it abstracts the build tool) are you using?

Next.js (details below)

What browser are you using?

Chrome latest 64-bit 89.0.etc

What operating system are you using?

Windows 10

Reproduction repository

https://github.com/DoctorDerek/steven-terner.github.io/blob/master/pages/demo.js


Hi, I'm crossposting this from tailwindlabs/tailwindcss-typography#157 because it affects both libraries.


Following up on a comment from @danjac about tailwindlabs/tailwindcss-typography#153 -- there's a bug that I'm reporting to jit & typography.

Here's an example repo that uses dark theme with prose / Tailwind typography: https://github.com/timlrx/tailwind-nextjs-starter-blog/ (not jit) and it's live here: https://tailwind-nextjs-starter-blog.vercel.app/

As you can see, prose works fine with dark mode in the old system (except for the caveat about responsive classes addressed by that PR tailwindlabs/tailwindcss-typography#153 ):
image

However when I tried that in a derivative from that template using the new jit engine, prose doesn't work with dark mode.
image

Source code: https://github.com/DoctorDerek/steven-terner.github.io/blob/master/pages/demo.js Live version: http://steven-terner-github-io.vercel.app/demo (That page won't stay up for particularly long, but I didn't want to spin up a new project just for this issue, sorry.)

 <article className="mx-auto space-y-6 prose">
        <SectionContainer>
          <p>
            My portfolio showcases 

I'm happy to look into it more myself if you'd like to assign me to following up on it, but now I'm just the messenger 😀

@DoctorDerek
Copy link
Author

I'm not sure if this is implementation-dependent or was fixed in a recent version change, but prose seems to work correctly with dark mode now. See timlrx/tailwind-nextjs-starter-blog#14 live at https://tailwind-nextjs-starter-blog-git-upgrade-timlrx.vercel.app/

Happy to re-open if it's a lingering issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant