Solve_issue_868 (#870)

* Issue correction # 868 by using the drop event to select the target row * Update CHANGES.rst * Update CHANGES.rst User in a new line. * Update layout.js Fixed indentation using 2 spaces. * Update CHANGES.rst Fixed identation
collective · Oct 2, 2019 · 588edf1 · 588edf1
1 parent 8b3c5c8
commit 588edf1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -6,7 +6,11 @@ There's a frood who really knows where his towel is.
 2.2.1 (unreleased)
 ^^^^^^^^^^^^^^^^^^
 
-- The 'drop' event and its 'target' method are used to obtain the element (column) and not generate ambiguity (fixes `#861 <https://github.com/collective/collective.cover/issues/861>`_). [Mubra]
+- A "row" object is generated and its location in the DOM is added with the even "drop.target" (fixes `#868 <https://github.com/collective/collective.cover/issues/868>`_).
+  [Mubra]
+
+- The 'drop' event and its 'target' method are used to obtain the element (column) and not generate ambiguity (fixes `#861 <https://github.com/collective/collective.cover/issues/861>`_).
+  [Mubra]
 
 
 2.2.0 (2019-02-26)

diff --git a/webpack/app/js/layout.js b/webpack/app/js/layout.js
@@ -236,7 +236,10 @@ export default class LayoutView {
 
     //allow columns droppable
     let onDrop = function(e, ui) {
-      this.row_drop(rows);
+      //the origin row is taken with the "drop" event
+      let helprows = this.row_dom.clone();
+      helprows['0']=e.target;
+      this.row_drop(helprows);
     };
     rows.droppable({
       activeClass: 'ui-state-default',