Reduce script loads to one load per script

[boazsender]

It seems like the same scripts p5.min.js, p5.dom.min.js, p5.sound.min.js, loop-protect.min.js, hijackConsole.js are reloaded everytime the play button is pressed:

This should not be required, and loading them once would improve performance.

[jmandel1027]

If no one has looked into this I can totally work on a patch this weekend n_n

[catarak]

@jay-manday go ahead and work on this!

[jmandel1027]

totally! I have a feeeeeling that it's its coming from the files.js reducer thats injecting the sketch into the editor here:

https://github.com/processing/p5.js-web-editor/blob/fd284358c2a1b361e5cfaf1463389a06ce8b28c1/client/modules/IDE/reducers/files.js

I'll keep ya posted! Btw can you hook me up with your s3 buckets? was a little unsure about how to set mine up.

[ashu8912]

Sending in a pr that fixes this

[ashu8912]

Read how the PreviewFrame is working and got to a conclusion
1.It uses an iframe to show the preview
2.That iframe is in itself like a mini browser where we add the html and all references to the script
3.Now The Thing is each time the play button is clicked that template for html is recreated and that is the reason why each time a new request is sent for the p5.js libraries.

Couldn't figure it out how will i prevent these scripts from getting refetched coz the iframe is like a blackbox where we have got no control.

@catarak would like to hear your views on this one