fix: fix the object will be copied is not passed to the clone function & set a default value for zindex

ly525 · ly525 · commit 2de1aa8cfd49 · 2019-10-22T08:32:43.000+08:00
diff --git a/front-end/h5/src/components/core/models/element.js b/front-end/h5/src/components/core/models/element.js
@@ -107,7 +107,7 @@ class Element {
     return data
   }
 
-  clone ({ zindex }) {
+  clone ({ zindex = this.commonStyle.zindex + 1 } = {}) {
     return new Element({
       zindex,
       name: this.name,
diff --git a/front-end/h5/src/store/modules/element.js b/front-end/h5/src/store/modules/element.js
@@ -52,7 +52,7 @@ export const mutations = {
         elements.push(element)
         break
       case 'copy':
-        elements.push(state.editingElement.clone())
+        elements.push(state.editingElement.clone({ zindex: len + 1 }))
         break
       case 'delete':
         {

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ class Element {`
`107`	`107`	`return data`
`108`	`108`	`}`
`109`	`109`
`110`		`- clone ({ zindex }) {`
	`110`	`+ clone ({ zindex = this.commonStyle.zindex + 1 } = {}) {`
`111`	`111`	`return new Element({`
`112`	`112`	`zindex,`
`113`	`113`	`name: this.name,`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ export const mutations = {`
`52`	`52`	`elements.push(element)`
`53`	`53`	`break`
`54`	`54`	`case 'copy':`
`55`		`- elements.push(state.editingElement.clone())`
	`55`	`+ elements.push(state.editingElement.clone({ zindex: len + 1 }))`
`56`	`56`	`break`
`57`	`57`	`case 'delete':`
`58`	`58`	`{`