-
Notifications
You must be signed in to change notification settings - Fork 13
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
Replace Ace with CodeMirror #23
Comments
I'm going to merge with #22 before I do a PR for this |
Just for completeness sake, can we document here the reasoning for switching editors? (I don't have much experience with either, so this is partly because I'm curious to learn the pros/cons.) |
@makenai Did you have any specific preference for one over the other? There is an npm package for CodeMirror which is nice. Googling is showing points in favor of both so I don't know that there's a clear winner. This is an interesting thread: adobe/brackets#1142 |
I have a lot of experience with CodeMirror... I use it in my dayjob project. It is solid. I don't know anything about Ace, but from my understanding, it is equally solid. |
CodeMirror was used by j5-chrome for the original Chromebots too. |
Don't remember why we switched on chromebots, but I'm really liking codemirror |
Removing the Ace library from static and adding the CodeMirror npm module to robotnik.js
The text was updated successfully, but these errors were encountered: