9828377f157cb25b239a4dd95e218e23373aed16 Fix: Sizing table was incorr…

…ectly including DataTables hidden columns

Sync to source repo @9828377f157cb25b239a4dd95e218e23373aed16

datatables.json

@@ -11,5 +11,5 @@
     ],
     "src-repo": "http://github.com/DataTables/Responsive",
     "last-tag": "3.0.1",
-    "last-sync": "5cef001d85ad066031fb4febbbca666f9eadc533"
+    "last-sync": "9828377f157cb25b239a4dd95e218e23373aed16"
 }

js/dataTables.responsive.js

@@ -1138,8 +1138,7 @@ $.extend(Responsive.prototype, {
 			emptyRow.append('<td/>');
 		}
 
-		// Body rows - we don't need to take account of DataTables' column
-		// visibility since we implement our own here (hence the `display` set)
+		// Body rows
 		dt.rows({ page: 'current' }).every(function (rowIdx) {
 			var node = this.node();
 
@@ -1150,7 +1149,7 @@ $.extend(Responsive.prototype, {
 			// We clone the table's rows and cells to create the sizing table
 			var tr = node.cloneNode(false);
 
-			dt.cells(rowIdx, '*').every(function (rowIdx2, colIdx) {
+			dt.cells(rowIdx, visibleColumns).every(function (rowIdx2, colIdx) {
 				// If nodes have been moved out (listHiddenNodes), we need to
 				// clone from the store
 				var store = that.s.childNodeStore[rowIdx + '-' + colIdx];
@@ -1168,6 +1167,8 @@ $.extend(Responsive.prototype, {
 			clonedBody.append(tr);
 		});
 
+		// Any cells which were hidden by Responsive in the host table, need to
+		// be visible here for the calculations
 		clonedBody.find('th, td').css('display', '');
 
 		// Footer

js/dataTables.responsive.mjs

@@ -1098,8 +1098,7 @@ $.extend(Responsive.prototype, {
 			emptyRow.append('<td/>');
 		}
 
-		// Body rows - we don't need to take account of DataTables' column
-		// visibility since we implement our own here (hence the `display` set)
+		// Body rows
 		dt.rows({ page: 'current' }).every(function (rowIdx) {
 			var node = this.node();
 
@@ -1110,7 +1109,7 @@ $.extend(Responsive.prototype, {
 			// We clone the table's rows and cells to create the sizing table
 			var tr = node.cloneNode(false);
 
-			dt.cells(rowIdx, '*').every(function (rowIdx2, colIdx) {
+			dt.cells(rowIdx, visibleColumns).every(function (rowIdx2, colIdx) {
 				// If nodes have been moved out (listHiddenNodes), we need to
 				// clone from the store
 				var store = that.s.childNodeStore[rowIdx + '-' + colIdx];
@@ -1128,6 +1127,8 @@ $.extend(Responsive.prototype, {
 			clonedBody.append(tr);
 		});
 
+		// Any cells which were hidden by Responsive in the host table, need to
+		// be visible here for the calculations
 		clonedBody.find('th, td').css('display', '');
 
 		// Footer