Skip to content
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

[CLOSED] Ace? #1129

Open
core-ai-bot opened this issue Aug 29, 2021 · 3 comments
Open

[CLOSED] Ace? #1129

core-ai-bot opened this issue Aug 29, 2021 · 3 comments

Comments

@core-ai-bot
Copy link
Member

Issue by bootstraponline
Monday Jun 25, 2012 at 22:25 GMT
Originally opened as adobe/brackets#1142


Is there any documentation about why brackets selected CodeMirror instead of Ace? I'm interested in the technical reasons that led to selecting one over the other. Thanks.

@core-ai-bot
Copy link
Member Author

Comment by adrocknaphobia
Monday Jun 25, 2012 at 23:04 GMT


Our original prototype ran completely in the browser and was powered by the ACE editor. ACE editor is great and we even contributed some bug fixes back to the project. However, we wanted to do our due diligence and also researched using CodeMirror.

In the end we choose CodeMirror because it is licensed under MIT and that is the license we ultimately wanted for Brackets. We've been working closely with Marijn, the creator of CodeMirror, and have been submitting pull requests back to the main project. Our fork of CodeMirror is public and we plan to submit all our changes back to the main project (including the Quick Edit functionality).

@core-ai-bot
Copy link
Member Author

Comment by mrchief
Wednesday Jun 27, 2012 at 15:04 GMT


ACE works on IE as well which CodeMirror doesn't support. Besides, CodeMirror seems to have weird copy/paste issues (based on experience with jsFiddle).

Other than that, ACE is lot better performing in many aspects but I guess you guys have done your research (CM 2.0 was supposed to have many features and improvments that ACE currently has).

Not major issues per se and of course, when enable ti for browsers, IE would be left out I guess (abandoning IE seems to be a (lazy) fashion statement now-a-days)!

@core-ai-bot
Copy link
Member Author

Comment by bootstraponline
Saturday Sep 29, 2012 at 19:25 GMT


ACE is now licensed under the New BSD License.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant