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

Moving into Next.js examples repo? #2

Closed
leerob opened this issue May 26, 2023 · 1 comment · Fixed by vercel/next.js#50394
Closed

Moving into Next.js examples repo? #2

leerob opened this issue May 26, 2023 · 1 comment · Fixed by vercel/next.js#50394

Comments

@leerob
Copy link

leerob commented May 26, 2023

Would you be open to moving this into the Next.js examples repo as a PR? (And updating the deps, removing the experimental flag)?

@SuttonJack
Copy link
Owner

SuttonJack commented May 26, 2023

@leerob vercel/next.js#50394

Note that the experimental flag is required at the moment for the VE plugin - see:

  1. Next.js 13 - experimental.appDir still being used in stable App Router vanilla-extract-css/vanilla-extract#1087
  2. Plugin support for Vanilla Extract vercel/next.js#49892 (comment)

Ideally we can work this out in the near future

leerob pushed a commit to vercel/next.js that referenced this issue May 27, 2023
### What?
- Adds an updated Vanilla Extract example
- ~~Note that `@vanilla-extract/next-plugin` currently requires
`experimental: { appDir: true }` to be set.~~

### Why?
- The
[documentation](https://nextjs.org/docs/app/building-your-application/styling/css-in-js)
currently links to a [personal
repo](https://github.com/SuttonJack/vanilla-extract-app-dir)
- Closes SuttonJack/vanilla-extract-app-dir#2

<p align="center"><img width="600"
src="https://github.com/vercel/next.js/assets/92991945/eeab322e-e75e-4999-9fe7-39aa15c45c11"
/></p>

-->
hydRAnger pushed a commit to hydRAnger/next.js that referenced this issue Jun 12, 2023
### What?
- Adds an updated Vanilla Extract example
- ~~Note that `@vanilla-extract/next-plugin` currently requires
`experimental: { appDir: true }` to be set.~~

### Why?
- The
[documentation](https://nextjs.org/docs/app/building-your-application/styling/css-in-js)
currently links to a [personal
repo](https://github.com/SuttonJack/vanilla-extract-app-dir)
- Closes SuttonJack/vanilla-extract-app-dir#2

<p align="center"><img width="600"
src="https://github.com/vercel/next.js/assets/92991945/eeab322e-e75e-4999-9fe7-39aa15c45c11"
/></p>

-->
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 a pull request may close this issue.

2 participants