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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add es6 module support for addons/modes #5403
Comments
We're planning a modernization of the codebase in the near future, but until then I'd prefer to keep things stable and not add further complexity and indirection, so this isn't something that's welcome right now. |
@alangdm there's a workaround for some modes at least. You can define a module that imports CodeMirror then writes it to a global, and import that before any mode: codemirror-global.js: import CodeMirror from "codemirror/src/codemirror.js";
window.CodeMirror = CodeMirror; app.js: import './codemirror-global.js';
import CodeMirror from "codemirror/src/codemirror.js";
import "codemirror/mode/javascript/javascript.js";
Wouldn't this reduce complexity and indirection by directly using the web native module system, rather than relying on user-land loaders? We're getting a lot of demand for libraries like CodeMirror to be loadable as modules. What can we do to help? |
any update? @justinfagnani |
is there an up to date guide on how to use CodeMirror in an ES modules environment? |
No. But the gist of it would be to run something like |
Ye, I had totally missed codemirror 6. This makes way more sense |
If you're having trouble with the global CodeMirror workaround, try using the import CodeMirror from "codemirror/src/codemirror.js";
window.CodeMirror = CodeMirror;
import("codemirror/mode/javascript/javascript.js").then(() => {
console.log("imported CodeMirror 'javascript' mode");
}); |
Edit: I found out about the discussion forum a tad to late, feel free to ignore this here 馃槗
Hi, I've been trying to use CodeMirror inside a WebComponent made with LitElement (demo here)and I somehow managed to make it work but there's a couple of things I hoped you could evaluate adding to make life easier for people developing on es6 modules like me.
Since the CodeMirror core src folder is on npm and it's already in modules it can be used just fine like that in other modules, but there's a bit of a problem when trying to use any of the addons/modes provided here;
As far as I can tell, all addons/modes all asume that if you're not using CommonJS/AMD you must be using a global script, but since es6 modules have local environments trying to import the addons/modes as they are right now in an es6 module ends up with a
CodeMirror is not defined
error, e.g.:For now, I've gotten this to work by using a function which fetches the addon/mode and then
eval
s it in the module's context but that is far from ideal thinking of the risks and debugging troubles usingeval
brings, so I was hoping you could release an es6 modules version of the addons/modes, it could be something as simple as changing:If you like this way of doing things I can make a PR for this kind of behavior.
Now that Firefox joined every other major modern browser in supporting es6 modules I think it could be a great time to add this 馃憤
The text was updated successfully, but these errors were encountered: