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

Migrate to ACE editor #574

Merged
merged 4 commits into from Feb 26, 2015
Merged

Migrate to ACE editor #574

merged 4 commits into from Feb 26, 2015

Conversation

knolleary
Copy link
Member

Replaces our use of Eclipse Orion with the ACE editor.

Updates to:

  • template, function and comment nodes
  • library browser

RED.editor.createEditor utility function added to make it easier to create an instance of ACE with suitable default values.

Orion code left in place for existing nodes that rely on us providing it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.53% when pulling bf1cd45 on knolleary:editor into 393fc34 on node-red:master.

knolleary added a commit that referenced this pull request Feb 26, 2015
@knolleary knolleary merged commit 172cbda into node-red:master Feb 26, 2015
@knolleary knolleary deleted the editor branch February 26, 2015 22:27
@maxandersen
Copy link

Can I ask for the reasons to move from orion to ace ? curious to know.

@knolleary
Copy link
Member Author

Hi @maxandersen

it was largely a pragmatic choice based on the relative maturity of both editor components.

We're big fans of Eclipse Orion. Their focus has been on the complete IDE environment, not just the editor component. We found it a bit of a struggle to add in some of the nicer functionality (code completion, error highlighting etc) to the Orion widget without seeming to need a lot of bits and pieces.

ACE was a much more self-contained thing that got us those advanced features much more easily.

Ultimately, we have limited resources and lots to do in Node-RED - so as I said, it was a pragmatic choice to get the functionality we needed.

I'm not at all opposed to moving back to Orion at some point in the future - in fact, part of this change included providing the RED.editor.createEditor function as a point of some-what-abstraction to hide the specifics of the editor code from a node that wants to use it.

@maxandersen
Copy link

Thanks for the answer - much appreciated.

I was just looking at node-red because it was integrating orion and then got surprised when I found it moved off so was curious what caused it.

Full IDE vs "just an editor" explains it.

@dceejay
Copy link
Member

dceejay commented Apr 9, 2015

Hi Max, as Nick said the embedded version of Ace has a lot more in it than the embedded version of Orion. And indeed some of the features in the Orion one (like folding) just didn't work and had been an open issue for almost a year, so it was a bit of a no-brainer until it catches up / leapfrogs.

@maxandersen
Copy link

Do you have a link to the open issue about folding ? (since i'm using folding in orion and haven't noticed a problem)

@dceejay
Copy link
Member

dceejay commented Apr 9, 2015

Not a problem on the full version... but the embedded one wouldn't allow language support.
https://bugs.eclipse.org/bugs/show_bug.cgi?id=409800
opened June 2013 ...

@maxandersen
Copy link

Thanks! good to know.

any discussions raised on orion-dev on this ? when googling for it I don't find anything raised/discussed anywhere ;/

@dceejay
Copy link
Member

dceejay commented Apr 9, 2015

don't know. I did ping Grant in Jan, before we switched, and he confirmed it was still "on the list" to be fixed... so - life's too short to worry about it too much.

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

Successfully merging this pull request may close these issues.

None yet

4 participants