fix: do not copy canvas context if it has neither width nor height (#530)

swac · Claudéric Demers · commit 3808437d4c3a · 2019-04-23T20:29:11.000-04:00
diff --git a/src/utils.js b/src/utils.js
@@ -296,7 +296,11 @@ export function cloneNode(node) {
       field.name = `__sortableClone__${field.name}`;
     }
 
-    if (field.tagName === NodeType.Canvas) {
+    if (
+      field.tagName === NodeType.Canvas &&
+      fields[i].width > 0 &&
+      fields[i].height > 0
+    ) {
       const destCtx = field.getContext('2d');
       destCtx.drawImage(fields[i], 0, 0);
     }

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,11 @@ export function cloneNode(node) {`
`296`	`296`	field.name = `__sortableClone__${field.name}`;
`297`	`297`	`}`
`298`	`298`
`299`		`- if (field.tagName === NodeType.Canvas) {`
	`299`	`+ if (`
	`300`	`+ field.tagName === NodeType.Canvas &&`
	`301`	`+ fields[i].width > 0 &&`
	`302`	`+ fields[i].height > 0`
	`303`	`+ ) {`
`300`	`304`	`const destCtx = field.getContext('2d');`
`301`	`305`	`destCtx.drawImage(fields[i], 0, 0);`
`302`	`306`	`}`