Skip to content

Commit

Permalink
Remove fnConstruct in mouseUp
Browse files Browse the repository at this point in the history
The fnConstruct function in mouseUp caused some trouble when a state was loaded, columns where moved randomly. After it was removed I didn't notice any problems. 

It also resets the _ColReorder_iOrigCol which caused some problems when saving the state (the col Reorder was always sequential lik [1,2,3,4 etc.]
  • Loading branch information
jurruh authored Apr 10, 2019
1 parent 9ce30c6 commit f268336
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions ColReorderWithResize.js
Original file line number Diff line number Diff line change
Expand Up @@ -1315,9 +1315,6 @@ $.extend( ColReorder.prototype, {
this.s.dt.oInstance.fnAdjustColumnSizing( false );
}

// Re-initialize so as to register the new column order (otherwise the events remain bound to the original column indices).
this._fnConstruct();

this.s.dt.oInstance.trigger('column-reorder.dt.mouseup', [ this.s.dt ] );

/* Save the state */
Expand Down

0 comments on commit f268336

Please sign in to comment.