Skip to content
This repository was archived by the owner on May 16, 2026. It is now read-only.

Let's support Tailwind V4 - #87

Closed
cxa wants to merge 2 commits into
merrickluo:masterfrom
cxa:tailwindv4
Closed

cxa wants to merge 2 commits into
merrickluo:masterfrom
cxa:tailwindv4

Conversation

@cxa

@cxa cxa commented Aug 21, 2025 •

Copy link
Copy Markdown
Contributor

Since v4, the config file is optional.

This PR checks tailwindcss version from packages.json in workspace root, if version is larger than 3, we activate lsp-tailwindcss.

@cxa
cxa force-pushed the tailwindv4 branch 2 times, most recently from bc2fa02 to 0341603 Compare September 22, 2025 06:52
@merrickluo

Copy link
Copy Markdown
Owner

hi @cxa thanks for the idea and the pull request, I used a slight different approach in order to:

  • remove the dependencies (f.el, cl-lib)
  • make it work with projects that don't have package.json on workspace root

I'll close this one in favor of #88, thanks again! 👍

@merrickluo merrickluo closed this Oct 5, 2025
@cxa

cxa commented Oct 5, 2025

Copy link
Copy Markdown
Contributor Author

No problem. Nice to see you catching up.

P.S. I've switched to Eglot w/ lspx and use some cache mechanism to enhance auto completion experience because the Tailwind LSP responses more than 10k items in common cases, Emacs 30+ requires more than 2 seconds to parse.

@merrickluo

Copy link
Copy Markdown
Owner

oh nice, I've always wanted to migrate to eglot too, glad to know lspx works! def will try that later.

@cxa

cxa commented Oct 7, 2025

Copy link
Copy Markdown
Contributor Author

oh nice, I've always wanted to migrate to eglot too, glad to know lspx works! def will try that later.

My solution if you're interested:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants