Change default formatter settings for LaTeX (#28727)
Closes: https://github.com/rzukic/zed-latex/issues/77 ## Default formatter: `latexindent` Before, formatting was delegated to the language server, which just ran a `latexindent` executable. There was no benefit to running it through the language server over running it as an "external" formatter in zed. In fact this was an issue because there was no way to provide an explicit path for the executable (causing above extension issue). Having the default settings configure the formatter directly gives more control to user and removes the number of indirections making it clearer how to tweak things like the executable path, or extra CLI args, etc... ## Alternative: `prettier` Default settings have also been added to allow prettier as the formatter (by just setting `"formatter": "prettier"` in the "LaTeX" language settings). This was not possible before because an extra line needed to be added to the `prettier` crate (similarly to what was done for https://github.com/zed-industries/zed/issues/19024) to find the plugin correctly. > [!NOTE] > The `prettier-plugin-latex` node module also contained a `dist/standalone.js` but using that instead of `dist/prettier-plugin-latex.js` gave an error, and indeed the latter worked as intended (along with its questionable choices for formatting). Release Notes: - LaTeX: added default `latexindent` formatter settings without relying on `texlab`, as well as allowing `prettier` to be chosen for formatting --------- Co-authored-by: Peter Tripp <peter@zed.dev>
This commit is contained in:
parent
5ae8c4cf09
commit
522abe8e59
2 changed files with 4 additions and 2 deletions
|
@ -1497,11 +1497,11 @@
|
|||
}
|
||||
},
|
||||
"LaTeX": {
|
||||
"format_on_save": "on",
|
||||
"formatter": "language_server",
|
||||
"language_servers": ["texlab", "..."],
|
||||
"prettier": {
|
||||
"allowed": false
|
||||
"allowed": true,
|
||||
"plugins": ["prettier-plugin-latex"]
|
||||
}
|
||||
},
|
||||
"Markdown": {
|
||||
|
|
|
@ -343,6 +343,8 @@ impl Prettier {
|
|||
prettier_plugin_dir.join("plugin.js"),
|
||||
// this one is for @prettier/plugin-php
|
||||
prettier_plugin_dir.join("standalone.js"),
|
||||
// this one is for prettier-plugin-latex
|
||||
prettier_plugin_dir.join("dist").join("prettier-plugin-latex.js"),
|
||||
prettier_plugin_dir,
|
||||
]
|
||||
.into_iter()
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue