Add allow-modals to iframe, allowing alert and prompt. Discussed in Issue processing#507. (processing#518)

Chris Hallberg · catarak · commit 62670e6dda3d · 2018-02-07T12:47:25.000-05:00
diff --git a/client/modules/IDE/components/PreviewFrame.jsx b/client/modules/IDE/components/PreviewFrame.jsx
@@ -407,7 +407,7 @@ class PreviewFrame extends React.Component {
         frameBorder="0"
         title="sketch output"
         ref={(element) => { this.iframeElement = element; }}
-        sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-forms"
+        sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-forms allow-modals"
       />
     );
   }

Original file line number	Diff line number	Diff line change
`@@ -407,7 +407,7 @@ class PreviewFrame extends React.Component {`
`407`	`407`	`frameBorder="0"`
`408`	`408`	`title="sketch output"`
`409`	`409`	`ref={(element) => { this.iframeElement = element; }}`
`410`		`- sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-forms"`
	`410`	`+ sandbox="allow-scripts allow-pointer-lock allow-same-origin allow-popups allow-forms allow-modals"`
`411`	`411`	`/>`
`412`	`412`	`);`
`413`	`413`	`}`