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

fix: stylex example to use new vite plugin #1105

Merged
merged 2 commits into from
Dec 28, 2024

Conversation

tylersayshi
Copy link
Contributor

@tylersayshi tylersayshi commented Dec 28, 2024

fixes #750

Copy link

vercel bot commented Dec 28, 2024

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
waku ⬜️ Ignored (Inspect) Visit Preview Dec 28, 2024 8:18pm

Copy link

codesandbox-ci bot commented Dec 28, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Contributor

@himself65 himself65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my friend in a telegram cn frontend group is maintaining vite-plugin-stylex-dev, I think he is not actively working on this now? @nonzzz

He has no message since May 2024 🤔

Copy link
Owner

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this!

@dai-shi dai-shi merged commit a7b2566 into dai-shi:main Dec 28, 2024
26 checks passed
@dai-shi
Copy link
Owner

dai-shi commented Dec 29, 2024

There seems an issue with pnpm -F 37_css dev.

first load

image

after reload

image

Can you reproduce it on your end? @tylersayshi

@tylersayshi
Copy link
Contributor Author

@dai-shi i see the same. I might not have time to debug tonight but could look closer soon.

This is specific to dev though. The issue is not present once we run waku build and waku start

@dai-shi
Copy link
Owner

dai-shi commented Dec 29, 2024

No hurry. Yeah, dev only, but that's the issue. I think that's why we changed the plugin to vite-plugin-stylex-dev, or maybe.

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.

styles are not applied correctly with stylex in server components
3 participants