From 32c54b85f3e1cf17526013119376595604373f84 Mon Sep 17 00:00:00 2001
From: Mottie <wowmotty@gmail.com>
Date: Mon, 5 Jan 2015 17:44:26 -0600
Subject: [PATCH 1/4] Only display visible rows while moving

---
 jquery.dragtable.js | 30 ++++++++++++++++--------------
 1 file changed, 16 insertions(+), 14 deletions(-)

diff --git a/jquery.dragtable.js b/jquery.dragtable.js
index 32a4f23..44d0192 100644
--- a/jquery.dragtable.js
+++ b/jquery.dragtable.js
@@ -115,7 +115,7 @@
     },
     // bubble the moved col left or right
     _bubbleCols: function() {
-      var i, j, col1, col2;
+      var i, j, col1, col2, visibleRows;
       var from = this.originalTable.startIndex;
       var to = this.originalTable.endIndex;
       /* Find children thead and tbody.
@@ -125,22 +125,23 @@
       if (this.options.excludeFooter) {
         thtb = thtb.not('tfoot');
       }
+      visibleRows = thtb.find('> tr:visible');
       if (from < to) {
         for (i = from; i < to; i++) {
-          col1 = thtb.find('> tr > td:nth-child(' + i + ')')
-            .add(thtb.find('> tr > th:nth-child(' + i + ')'));
-          col2 = thtb.find('> tr > td:nth-child(' + (i + 1) + ')')
-            .add(thtb.find('> tr > th:nth-child(' + (i + 1) + ')'));
+          col1 = visibleRows.find('> td:nth-child(' + i + ')')
+            .add(visibleRows.find('> th:nth-child(' + i + ')'));
+          col2 = visibleRows.find('> td:nth-child(' + (i + 1) + ')')
+            .add(visibleRows.find('> th:nth-child(' + (i + 1) + ')'));
           for (j = 0; j < col1.length; j++) {
             swapNodes(col1[j], col2[j]);
           }
         }
       } else {
         for (i = from; i > to; i--) {
-          col1 = thtb.find('> tr > td:nth-child(' + i + ')')
-            .add(thtb.find('> tr > th:nth-child(' + i + ')'));
-          col2 = thtb.find('> tr > td:nth-child(' + (i - 1) + ')')
-            .add(thtb.find('> tr > th:nth-child(' + (i - 1) + ')'));
+          col1 = visibleRows.find('> td:nth-child(' + i + ')')
+            .add(visibleRows.find('> th:nth-child(' + i + ')'));
+          col2 = visibleRows.find('> td:nth-child(' + (i - 1) + ')')
+            .add(visibleRows.find('> th:nth-child(' + (i - 1) + ')'));
           for (j = 0; j < col1.length; j++) {
             swapNodes(col1[j], col2[j]);
           }
@@ -196,7 +197,7 @@
       var rowAttrsArr = [];
       //compute height, special handling for ie needed :-(
       var heightArr = [];
-      this.originalTable.el.find('tr').slice(0, this.options.maxMovingRows).each(function(i, v) {
+      this.originalTable.el.find('tr:visible').slice(0, this.options.maxMovingRows).each(function(i, v) {
         // row attributes
         var attrs = this.attributes;
         var attrsString = "";
@@ -220,7 +221,8 @@
       if (this.options.excludeFooter) {
         thtb = thtb.not('tfoot');
       }
-      thtb.find('> tr > th').each(function(i, v) {
+      var visibleRows = thtb.find('> tr:visible');
+      visibleRows.find('> th').each(function(i, v) {
         var w = $(this).outerWidth();
         widthArr.push(w);
         totalWidth += w;
@@ -234,13 +236,13 @@
 
       var sortableHtml = '<ul class="dragtable-sortable" style="position:absolute; width:' + totalWidth + 'px;">';
       // assemble the needed html
-      thtb.find('> tr > th').each(function(i, v) {
+      visibleRows.find('> th').each(function(i, v) {
         var width_li = $(this).outerWidth();
         sortableHtml += '<li style="width:' + width_li + 'px;">';
         sortableHtml += '<table ' + attrsString + '>';
-        var row = thtb.find('> tr > th:nth-child(' + (i + 1) + ')');
+        var row = visibleRows.find('> th:nth-child(' + (i + 1) + ')');
         if (_this.options.maxMovingRows > 1) {
-          row = row.add(thtb.find('> tr > td:nth-child(' + (i + 1) + ')').slice(0, _this.options.maxMovingRows - 1));
+          row = row.add(visibleRows.find('> td:nth-child(' + (i + 1) + ')').slice(0, _this.options.maxMovingRows - 1));
         }
         row.each(function(j) {
           // TODO: May cause duplicate style-Attribute

From 669d2f3e2a5c196db65e6ed27845dc8c2d5c848b Mon Sep 17 00:00:00 2001
From: Mottie <wowmotty@gmail.com>
Date: Mon, 5 Jan 2015 18:07:00 -0600
Subject: [PATCH 2/4] Include matched-height caption placeholder in movable
 columns

---
 jquery.dragtable.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/jquery.dragtable.js b/jquery.dragtable.js
index 44d0192..088cb99 100644
--- a/jquery.dragtable.js
+++ b/jquery.dragtable.js
@@ -234,12 +234,18 @@
       // one extra px on right and left side
       totalWidth += 2
 
+      var captionHeight = 0;
+      _this.originalTable.el.find('caption').each(function(){
+        captionHeight += $(this).outerHeight();
+      });
+
       var sortableHtml = '<ul class="dragtable-sortable" style="position:absolute; width:' + totalWidth + 'px;">';
       // assemble the needed html
       visibleRows.find('> th').each(function(i, v) {
         var width_li = $(this).outerWidth();
         sortableHtml += '<li style="width:' + width_li + 'px;">';
         sortableHtml += '<table ' + attrsString + '>';
+        sortableHtml += captionHeight ? '<caption style="height:' + captionHeight + 'px"></caption>' : '';
         var row = visibleRows.find('> th:nth-child(' + (i + 1) + ')');
         if (_this.options.maxMovingRows > 1) {
           row = row.add(visibleRows.find('> td:nth-child(' + (i + 1) + ')').slice(0, _this.options.maxMovingRows - 1));

From 3e69a3a7d42e3ac229e556700fc0dfa6ab35214d Mon Sep 17 00:00:00 2001
From: Mottie <wowmotty@gmail.com>
Date: Mon, 12 Jan 2015 15:05:11 -0600
Subject: [PATCH 3/4] Swap all rows, not just visible ones - oops!

---
 jquery.dragtable.js | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/jquery.dragtable.js b/jquery.dragtable.js
index 088cb99..2700849 100644
--- a/jquery.dragtable.js
+++ b/jquery.dragtable.js
@@ -125,23 +125,23 @@
       if (this.options.excludeFooter) {
         thtb = thtb.not('tfoot');
       }
-      visibleRows = thtb.find('> tr:visible');
+      row = thtb.find('> tr');
       if (from < to) {
         for (i = from; i < to; i++) {
-          col1 = visibleRows.find('> td:nth-child(' + i + ')')
-            .add(visibleRows.find('> th:nth-child(' + i + ')'));
-          col2 = visibleRows.find('> td:nth-child(' + (i + 1) + ')')
-            .add(visibleRows.find('> th:nth-child(' + (i + 1) + ')'));
+          col1 = row.find('> td:nth-child(' + i + ')')
+            .add(row.find('> th:nth-child(' + i + ')'));
+          col2 = row.find('> td:nth-child(' + (i + 1) + ')')
+            .add(row.find('> th:nth-child(' + (i + 1) + ')'));
           for (j = 0; j < col1.length; j++) {
             swapNodes(col1[j], col2[j]);
           }
         }
       } else {
         for (i = from; i > to; i--) {
-          col1 = visibleRows.find('> td:nth-child(' + i + ')')
-            .add(visibleRows.find('> th:nth-child(' + i + ')'));
-          col2 = visibleRows.find('> td:nth-child(' + (i - 1) + ')')
-            .add(visibleRows.find('> th:nth-child(' + (i - 1) + ')'));
+          col1 = row.find('> td:nth-child(' + i + ')')
+            .add(row.find('> th:nth-child(' + i + ')'));
+          col2 = row.find('> td:nth-child(' + (i - 1) + ')')
+            .add(row.find('> th:nth-child(' + (i - 1) + ')'));
           for (j = 0; j < col1.length; j++) {
             swapNodes(col1[j], col2[j]);
           }

From edb4047c6530355dba4ab3b7c40239e7f980af33 Mon Sep 17 00:00:00 2001
From: Mottie <wowmotty@gmail.com>
Date: Thu, 15 Jan 2015 06:40:57 -0600
Subject: [PATCH 4/4] add zIndex option

---
 jquery.dragtable.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/jquery.dragtable.js b/jquery.dragtable.js
index 2700849..c69b22a 100644
--- a/jquery.dragtable.js
+++ b/jquery.dragtable.js
@@ -74,7 +74,8 @@
       beforeStart: $.noop,         // returning FALSE will stop the execution chain.
       beforeMoving: $.noop,
       beforeReorganize: $.noop,
-      beforeStop: $.noop
+      beforeStop: $.noop,
+      zIndex: 10
     },
     originalTable: {
       el: null,
@@ -239,7 +240,7 @@
         captionHeight += $(this).outerHeight();
       });
 
-      var sortableHtml = '<ul class="dragtable-sortable" style="position:absolute; width:' + totalWidth + 'px;">';
+      var sortableHtml = '<ul class="dragtable-sortable" style="position:absolute; width:' + totalWidth + 'px; z-index:' + _this.options.zIndex + '">';
       // assemble the needed html
       visibleRows.find('> th').each(function(i, v) {
         var width_li = $(this).outerWidth();