We need the scope_opt_in_language_servers field so we can opt-in to tailwindcss-language-server in certain scopes.
The overrides work in a scope that has to be captured. In overrides.string the string is not the name of a syntactic node, but a tree-sitter capture variable. That means we need the overrides.scm file in which we capture all strings as @string. For some reason that's not 100% clear to me yet, we also need to add the other scopes that are used in the brackets configuration to the overrides. There's a specific piece of code in Zed that checks for this.
Sorry @ckipp01, we need a release for that again!
But the good news is that once this is merged and a new version released, the Tailwind server then works in Zed, even with - in completion names:
Turns out that #6 was missing two things:
scope_opt_in_language_serversfield so we can opt-in totailwindcss-language-serverin certain scopes.overrideswork in a scope that has to be captured. Inoverrides.stringthestringis not the name of a syntactic node, but a tree-sitter capture variable. That means we need theoverrides.scmfile in which we capture all strings as@string. For some reason that's not 100% clear to me yet, we also need to add the other scopes that are used in the brackets configuration to the overrides. There's a specific piece of code in Zed that checks for this.Sorry @ckipp01, we need a release for that again!
But the good news is that once this is merged and a new version released, the Tailwind server then works in Zed, even with
-in completion names:https://github.com/scalameta/metals-zed/assets/1185253/3d2857cd-8feb-4199-9217-56700063252a
Thanks to @nguyenyou's comment here: https://github.com/scalameta/metals-zed/pull/6#issuecomment-2116591990