-
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
Need option to ignore brace style #538
Comments
So, you want effectively a 'brace-style=none' option? How would that work? Just leave the braces where they are? By what definition? Interesting idea, but lower priority than a number of other feature requests. You should feel free to implement with tests and submit a pull request and we can discuss further. |
"where they are" === if on the same line (like collapse), leave them there but fix any whitespace I've started playing with the code and the first attempt (trying to hack code after reading existing code for 5 minutes) failed horribly, as expected. Will continue poking as I have time. 'brace-style=none' does sound better than 'brace-style=ignore' |
That seems to do the trick--never used npm test before and I can't get the original to work yet on my machine so still WIP |
Looks like a good start. |
Can we really implement When I use I hope to see this feature on https://github.com/victorporof/Sublime-HTMLPrettify |
If you see a problem, please file a new issue and reference this one. |
"Where to stick your stupid braces" is a common hot button for many teams so having to choose between collapse, expand, and end-expand will force us to either fight that battle or choose a different tool. Can we get an "ignore" option for -b | --brace-style, or else a separate --keep-brace-style option?
The text was updated successfully, but these errors were encountered: