Use a better beauitifer about brackets-beautify

[drewhamlett]

Hey @FezVrasta, we can definitely take a look at it, but I think you maybe getting hung up on too many edge cases. I've been using jsbeautify for years and it's been really good. It follows JSlint styling very well. They've always seem to put in the requests I submit.

I feel like your pain maybe with the css formatting anyway(as is mine) and I've already been looking around for other css formatters. I don't mind replacing the CSS and HTML formatter at any time. I'm all for that. I'm not for replacing the javascript formatter so swiftly though.

Jsbeautify has 5 years of tackling the edge cases where Pretty-Diff doesn't have that yet. That's 90% of the problems making a formatter.

I don't mind bringing in Pretty-Diff anyway because I've been trying to get basic php formatting support in. It looks pretty good. We could just bring it in, and maybe even have a formatting library selection. Or it could make sense as a separate extension. Not sure.

For JSbeautify, you may want to try to submit your request not as a pull request but a just as a feature request.

Let me know what you think. Thanks!

[FezVrasta]

Thanks,

yes I've huge problems with the CSS formatting using beautify-js, I definetely can not use it on my CSS and LESS files because it breaks too many things.

I can't neither try to beautify PHP templates because beautify-js doesn't support PHP, even if is just for some little variable inside the code.

I thought that use Pretty-Diff to beautify every kind of language could be easier than just use it for PHP and use the bugged beautify-js for the rest.

I'd like to develop a separated extension if you don't liekt o replace beautify-js. But I've not time and probably I'm not enough skilled to write this extension.

[ffrappo]

I'd like to back FezVrasta request, at least for CSS. Currently it's totally breaking all CSS pseudo selectors. Things will actually fail in the browser if the user doesn't notice and revert the "beautification".

[drewhamlett]

Ok guys, so I played around with the PrettyDIFF css formatter. While we may think that cssbeautify has a few issues, it doesn't have near the problems PrettyDIFF does. After testing it for a while it breaks on some basic things.

@FezVrasta JSbeautify did close your issue, beautify-web/js-beautify#363, and it seems to be great.

I pulled that in. You should download the github repo and test it out. I'm closing this for now, because it fixes the issue a bunch of people were having. We'll reopen this if there are more issues.

[ffrappo]

Awesome! Works like a charme now. Thanks guys.

[FezVrasta]

if the problem was just that one... it has lot of problems with LESS. With pretty Diff I can't really find any problem instead.

[bitwiseman]

@FezVrasta - please flie the issue you find. That's the only way the project will improve.