att / rcloud

Collaborative data analysis and visualization
http://rcloud.social
MIT License
430 stars 142 forks source link

monaco should only be loaded when merge activated #2641

Open gordonwoodhull opened 5 years ago

gordonwoodhull commented 5 years ago

For a quick fix to #2639, I put the require for the monaco editor at the global level of merge_view.js in aae72fa891659e6

This means that 2MB of code is loaded right when the edit.html page is loaded, and that code isn't used unless the user invokes Merge.