Core: Allow passing headers from multiple rows. See Mottie#1116

ams-tschoening · Apr 12, 2018 · 200fcac · 200fcac
1 parent 2983454
commit 200fcac
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/js/jquery.tablesorter.js b/js/jquery.tablesorter.js
@@ -518,11 +518,8 @@
 					ts.buildCache( c );
 				}
 				$cell = ts.getClosest( $( this ), '.' + ts.css.header );
-				// reference original table headers and find the same cell
-				// don't use $headers or IE8 throws an error - see #987
-				temp = $headers.index( $cell );
-				c.last.clickedIndex = ( temp < 0 ) ? $cell.attr( 'data-column' ) : temp;
-				// use column index if $headers is undefined
+				// use column index from data-attribute or index of current row; fixes #1116
+				c.last.clickedIndex = $cell.attr( 'data-column' ) || $cell.index();
 				cell = c.$headers[ c.last.clickedIndex ];
 				if ( cell && !cell.sortDisabled ) {
 					ts.initSort( c, cell, e );