From f268336f4daeb427b5db689d16ac5300f69d2cf4 Mon Sep 17 00:00:00 2001 From: Jurre Vriesen Date: Wed, 10 Apr 2019 13:58:38 +0200 Subject: [PATCH] Remove fnConstruct in mouseUp 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.] --- ColReorderWithResize.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/ColReorderWithResize.js b/ColReorderWithResize.js index 45c16a1..bfb037a 100644 --- a/ColReorderWithResize.js +++ b/ColReorderWithResize.js @@ -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 */