-
Notifications
You must be signed in to change notification settings - Fork 34
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
ES Modules #39
Comments
@lastmjs Hi I noticed your ES6-modules work-in-progress branch https://github.com/Juicy/juicy-ace-editor/compare/ES6-modules . Was there any issue with that? |
Hey, you can see where that went here: #41 I've handed over the reins. |
Oh, I see, thanks! |
Related -> ajaxorg/ace#3682 |
Using the es6 branch of this module with a bundler like Polymer's or Angular's I'm able to get syntax highlighting working by copying an ace build directories. This is makes for really messy looking builds but it works. |
It looks like Chrome is actually planning to remove HTML Import support soon. Pages with HTML Imports on have this console warning:
|
Would appreciate an update here as I'm not sure what is the recommended way to approach the Chrome deprecation warning |
Current work in progress PR: #51 |
Can we move the element to being loaded through an ES Module instead of HTML imports? I'm happy to do the work and make the PR if I can get the go-ahead
The text was updated successfully, but these errors were encountered: