Skip to content

Conversation

@jonas
Copy link
Collaborator

@jonas jonas commented Oct 2, 2018

The resulting code is not currently minified so the generated site is much larger. That should be fixed with the Webpack 4 upgrade in #105 .

@jonas jonas mentioned this pull request Oct 5, 2018
@jonas jonas force-pushed the monaco-editor-webpack-plugin branch from ae09f02 to 3bfdaa8 Compare October 5, 2018 17:47
@olafurpg
Copy link
Member

olafurpg commented Oct 7, 2018

#105 has now been merged so this needs a rebase. What would be the implications of loading monaco with the webpack plugin?

@jonas jonas force-pushed the monaco-editor-webpack-plugin branch from 3bfdaa8 to 154ea6d Compare October 8, 2018 04:59
@jonas
Copy link
Collaborator Author

jonas commented Oct 8, 2018

The plugin allows to only include the parts of the editor that is needed for this project, so ideally it will reduce the size.

This is currently blocked by microsoft/monaco-editor-webpack-plugin#26 which was fixed in 1.5.0 and requires Monaco Editor version 1.4.x.

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.

3 participants