Fix rebundling after every dep change

Ives van Hoorne · Ives van Hoorne · commit 42a6681920a5 · 2017-04-13T00:21:29.000+02:00
diff --git a/src/app/components/sandbox/Preview/index.js b/src/app/components/sandbox/Preview/index.js
@@ -110,7 +110,12 @@ export default class Preview extends React.PureComponent {
       this.state.frameInitialized
     ) {
       if (this.props.preferences.livePreviewEnabled) {
-        this.executeCode();
+        if (
+          this.props.bundle === prevProps.bundle || // So we don't trigger after every dep change
+          this.props.sandboxId !== prevProps.sandboxId
+        ) {
+          this.executeCode();
+        }
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,12 @@ export default class Preview extends React.PureComponent {`
`110`	`110`	`this.state.frameInitialized`
`111`	`111`	`) {`
`112`	`112`	`if (this.props.preferences.livePreviewEnabled) {`
`113`		`- this.executeCode();`
	`113`	`+ if (`
	`114`	`+ this.props.bundle === prevProps.bundle \|\| // So we don't trigger after every dep change`
	`115`	`+ this.props.sandboxId !== prevProps.sandboxId`
	`116`	`+ ) {`
	`117`	`+ this.executeCode();`
	`118`	`+ }`
`114`	`119`	`}`
`115`	`120`	`}`
`116`	`121`	`}`