[compoennt] adds clones type and formats comments

Wayne Van Son · Wayne Van Son · commit 6c1d5e576a93 · 2019-12-23T17:02:05.000+11:00
diff --git a/src/react-sortable.tsx b/src/react-sortable.tsx
@@ -74,7 +74,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     const { tag, style, className, id } = this.props;
     const classicProps = { style, className, id };
 
-    /** if no tag, default to a `div` element */
+    // if no tag, default to a `div` element.
     const newTag = !tag || tag === null ? "div" : tag;
     return createElement(
       newTag,
@@ -279,12 +279,10 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     return setList(newList, this.sortable, store);
   }
 
-  /** Called when the dragging starts */
   onStart(evt: SortableEvent) {
     store.dragging = this;
   }
 
-  /** Called when the dragging ends */
   onEnd(evt: SortableEvent) {
     store.dragging = null;
   }
@@ -303,7 +301,6 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  /** Called when the `onSpill` plugin is activated */
   onSpill(evt: SortableEvent) {
     const { removeOnSpill, revertOnSpill } = this.props;
     if (removeOnSpill && !revertOnSpill) removeNode(evt.item);
@@ -330,14 +327,16 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
   }
 }
 
-// everything below this point can be removes
+// everything below this point can be removed
 // once @types has been merged. PR submited
 interface MultiIndices {
   multiDragElement: HTMLElement;
   index: number;
 }
 
 export interface MultiDragEvent extends SortableEvent {
+  // @todo - add this to @types
+  clones: HTMLElement[];
   oldIndicies: MultiIndices[];
   newIndicies: MultiIndices[];
   swapItem: HTMLElement | null;

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl`
`74`	`74`	`const { tag, style, className, id } = this.props;`
`75`	`75`	`const classicProps = { style, className, id };`
`76`	`76`
`77`		- /** if no tag, default to a `div` element */
	`77`	+ // if no tag, default to a `div` element.
`78`	`78`	`const newTag = !tag \|\| tag === null ? "div" : tag;`
`79`	`79`	`return createElement(`
`80`	`80`	`newTag,`
`@@ -279,12 +279,10 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl`
`279`	`279`	`return setList(newList, this.sortable, store);`
`280`	`280`	`}`
`281`	`281`
`282`		`- /** Called when the dragging starts */`
`283`	`282`	`onStart(evt: SortableEvent) {`
`284`	`283`	`store.dragging = this;`
`285`	`284`	`}`
`286`	`285`
`287`		`- /** Called when the dragging ends */`
`288`	`286`	`onEnd(evt: SortableEvent) {`
`289`	`287`	`store.dragging = null;`
`290`	`288`	`}`
`@@ -303,7 +301,6 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl`
`303`	`301`	`setList(newList, this.sortable, store);`
`304`	`302`	`}`
`305`	`303`
`306`		- /** Called when the `onSpill` plugin is activated */
`307`	`304`	`onSpill(evt: SortableEvent) {`
`308`	`305`	`const { removeOnSpill, revertOnSpill } = this.props;`
`309`	`306`	`if (removeOnSpill && !revertOnSpill) removeNode(evt.item);`
`@@ -330,14 +327,16 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl`
`330`	`327`	`}`
`331`	`328`	`}`
`332`	`329`
`333`		`-// everything below this point can be removes`
	`330`	`+// everything below this point can be removed`
`334`	`331`	`// once @types has been merged. PR submited`
`335`	`332`	`interface MultiIndices {`
`336`	`333`	`multiDragElement: HTMLElement;`
`337`	`334`	`index: number;`
`338`	`335`	`}`
`339`	`336`
`340`	`337`	`export interface MultiDragEvent extends SortableEvent {`
	`338`	`+ // @todo - add this to @types`
	`339`	`+ clones: HTMLElement[];`
`341`	`340`	`oldIndicies: MultiIndices[];`
`342`	`341`	`newIndicies: MultiIndices[];`
`343`	`342`	`swapItem: HTMLElement \| null;`