From 45c6887ca2aeec30260bef587634ae6ab8ce2019 Mon Sep 17 00:00:00 2001
From: Artur Arseniev <artur.catch@hotmail.it>
Date: Thu, 10 Oct 2019 18:27:46 +0200
Subject: [PATCH] Handle canvas.scripts/canvas.styles options for each frame

---
 src/canvas/config/config.js  |  4 +++
 src/canvas/view/FrameView.js | 53 ++++++++++++++++++++++++++++++------
 2 files changed, 49 insertions(+), 8 deletions(-)

diff --git a/src/canvas/config/config.js b/src/canvas/config/config.js
index fb318b83b..1f5e49482 100644
--- a/src/canvas/config/config.js
+++ b/src/canvas/config/config.js
@@ -6,6 +6,8 @@ export default {
    * Be aware that these scripts will not be printed in the export code
    * @example
    * scripts: [ 'https://...1.js', 'https://...2.js' ]
+   * * // or passing objects as attributes
+   * scripts: [ { src: '/file.js', someattr: 'value' }, ... ]
    */
   scripts: [],
 
@@ -14,6 +16,8 @@ export default {
    * Be aware that these styles will not be printed in the export code
    * @example
    * styles: [ 'https://...1.css', 'https://...2.css' ]
+   * // or passing objects as attributes
+   * scripts: [ { href: '/style.css', someattr: 'value' }, ... ]
    */
   styles: [],
 
diff --git a/src/canvas/view/FrameView.js b/src/canvas/view/FrameView.js
index c8502cbbc..407fa7c47 100644
--- a/src/canvas/view/FrameView.js
+++ b/src/canvas/view/FrameView.js
@@ -1,5 +1,5 @@
 import Backbone from 'backbone';
-import { bindAll, isNumber, isNull, debounce } from 'underscore';
+import { bindAll, isNumber, isString, isNull, debounce } from 'underscore';
 import CssRulesView from 'css_composer/view/CssRulesView';
 import ComponentView from 'dom_components/view/ComponentView';
 import {
@@ -157,26 +157,63 @@ export default Backbone.View.extend({
   render() {
     const { el, $el, ppfx, config } = this;
     $el.attr({ class: ppfx + 'frame' });
-    if (config.renderContent) {
+
+    if (config.scripts.length) {
+      this.renderScripts();
+    } else if (config.renderContent) {
       el.onload = this.renderBody.bind(this);
     }
+
     return this;
   },
 
+  renderScripts() {
+    const { el, config } = this;
+    const appendScript = scripts => {
+      if (scripts.length > 0) {
+        const src = scripts.shift();
+        const scriptEl = createEl('script', {
+          type: 'text/javascript',
+          ...(isString(src) ? { src } : src)
+        });
+        scriptEl.onerror = scriptEl.onload = appendScript.bind(null, scripts);
+        el.contentDocument.head.appendChild(scriptEl);
+      } else {
+        this.renderBody();
+      }
+    };
+
+    el.onload = () => appendScript([...config.scripts]);
+  },
+
   renderBody() {
     const { config, model, ppfx } = this;
     const root = model.get('root');
     const styles = model.get('styles');
     const { em } = config;
     const doc = this.getDoc();
+    const head = this.getHead();
     const body = this.getBody();
     const conf = em.get('Config');
-
-    // Should be handled by `head`
-    // config.styles.forEach(style => {
-    //   externalStyles += `<link rel="stylesheet" href="${style}"/>`;
-    // });
-    // externalStyles && head.append(externalStyles);
+    const extStyles = [];
+
+    config.styles.forEach(href =>
+      extStyles.push(
+        isString(href)
+          ? {
+              tag: 'link',
+              attributes: { href, rel: 'stylesheet' }
+            }
+          : {
+              tag: 'link',
+              attributes: {
+                rel: 'stylesheet',
+                ...href
+              }
+            }
+      )
+    );
+    extStyles.length && appendVNodes(head, extStyles);
 
     const colorWarn = '#ffca6f';