fix #79 - incorrectly assuming that table cells are always going to b…

…e TD elements in the getSizingRow function used by IE
mkoryak · Apr 30, 2014 · e243ece · e243ece
1 parent 4a1951f
commit e243ece
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
diff --git a/dist/jquery.floatThead-slim.js b/dist/jquery.floatThead-slim.js
@@ -31,7 +31,7 @@
     getSizingRow: function($table, $cols, $fthCells){ // this is only called when using IE,
       // override it if the first row of the table is going to contain colgroups (any cell spans greater then one col)
       // it should return a jquery object containing a wrapped set of table cells comprising a row that contains no col spans and is visible
-      return $table.find('tbody tr:visible:first>td');
+      return $table.find('tbody tr:visible:first>*');
     },
     floatTableClass: 'floatThead-table',
     floatWrapperClass: 'floatThead-wrapper',
@@ -491,9 +491,9 @@
           if(haveCaption && captionAlignTop){
             tableOffset.top += captionHeight;
           }
-          var top, left, tableHeight;
+          var top, left;
+          var tableHeight = $table.outerHeight();
 
-          tableHeight = $table.outerHeight();
           if(locked && useAbsolutePositioning){ //inner scrolling, absolute positioning
             if (tableContainerGap >= scrollingContainerTop) {
               var gap = tableContainerGap - scrollingContainerTop;

diff --git a/dist/jquery.floatThead-slim.min.js b/dist/jquery.floatThead-slim.min.js
diff --git a/dist/jquery.floatThead.js b/dist/jquery.floatThead.js
@@ -31,7 +31,7 @@
     getSizingRow: function($table, $cols, $fthCells){ // this is only called when using IE,
       // override it if the first row of the table is going to contain colgroups (any cell spans greater then one col)
       // it should return a jquery object containing a wrapped set of table cells comprising a row that contains no col spans and is visible
-      return $table.find('tbody tr:visible:first>td');
+      return $table.find('tbody tr:visible:first>*');
     },
     floatTableClass: 'floatThead-table',
     floatWrapperClass: 'floatThead-wrapper',
@@ -491,9 +491,9 @@
           if(haveCaption && captionAlignTop){
             tableOffset.top += captionHeight;
           }
-          var top, left, tableHeight;
+          var top, left;
+          var tableHeight = $table.outerHeight();
 
-          tableHeight = $table.outerHeight();
           if(locked && useAbsolutePositioning){ //inner scrolling, absolute positioning
             if (tableContainerGap >= scrollingContainerTop) {
               var gap = tableContainerGap - scrollingContainerTop;