-
Notifications
You must be signed in to change notification settings - Fork 11
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
First version of formatting #3
Conversation
Thanks, I'll take a look at that when I get back home |
Conflicts: index.html
Auto formatting is great... but the issues with white spacing 😿 ... but with formatting, i think it also motives me more code mirror should be use, since it does support the diff plugin too. |
@gero3 i think we can merge this? we'll can deal with the white-spacing issue later... |
We should do that since I've been having problems with it before. I'll have to take my time to look at it. |
checking works great, thanks! would you be making a PR for the mdcs preset in node-jscs too? i'll merge this first and try adding code-mirror support... |
yep I'll create a PR in node-jscs. |
👍 😄 |
This is a first try at formatting the code.
It uses the latest version of jscs and the old code i had from months ago. It still has som minor issues but it works even on the webglrenderer file altough it takes a second or 2.
A preview is here:
https://preview.c9.io/gero3/mrdoobapproves/index.html