-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Format/Schema of .jsbeatifyrc #391
Comments
In my mind, it's a strong argument for spinning off the CSS and HTML beautifiers into separate projects. If the file was named ".beautifyrc", I could definitely see an argument for adding config for different types. |
👍 |
@evocateur I would be happy about that as well. Either support html/css fully and make them a first class citizen, like the suggested htm/css/js split of the config file. Or separate them into separate projects and possibly call the files .js/css/htmlbeautifyrc |
The discussion of separating the projects is issue #257. |
This has been address by having config file with settings per language and inheritance. |
This tool only supports .jsbeautifyrc files with the js options on the first level of the JSON file.
That was fine until this project got formatting for html/css as well.
Other projects
https://github.com/vitormil/sublime-text-2-preferences/tree/master/HTML-CSS-JS%20Prettify#using-your-own-jsbeautifyrc-options
and
https://github.com/vkadam/grunt-jsbeautifier#default-options-from-js-beautify-can-be-used
Uses one level to first separate html/css/js
Would be great if this structure could be used for this project as well. If doing
js-beautify style.css --type css
that results in in jsbeautify reading the config file and using what is the js settings for indent etc, not the css settings.The text was updated successfully, but these errors were encountered: