Merge pull request #1535 from ImagingDataCommons/release-50-sp

Table sizing bugfix

Author: s-paquette

static/js/tables.js

@@ -82,6 +82,10 @@ define(['cartutils','filterutils','tippy','jquery', 'base'], function(cartutils,
     });
     $('#proj_table, #cases_tab, #studies_tab, #series_tab, #cart-table').on('draw.dt', function(){
         window.hide_spinner();
+        // Auto-width needing to be shut off at the table header level during draw time.
+        $(this).children('th').each(function() {
+            this.style.width = null;
+        });
     });
 
     // Update the rows in the Projects Table, clear the other tables.
@@ -456,7 +460,6 @@ define(['cartutils','filterutils','tippy','jquery', 'base'], function(cartutils,
         });
     }
 
-
     window.getProjectCartStats = function(projidArr){
         var parsedFiltObj = filterutils.parseFilterObj();
         parsedFiltObj.collection_id = projidArr;
@@ -756,13 +759,13 @@ define(['cartutils','filterutils','tippy','jquery', 'base'], function(cartutils,
             updatePromise.reject();
         }
 
-        $('#cases_tab').on('draw.dt', function(){
-            $('#cases_table_head').children('tr').children().each(function(){
-                this.style.width=null;
-                }
-            );
+        // Remove autowidthing from the table header, which is reapplied any time it's created
+        $('#cases_table_head').children('tr').children().each(function(){
+            this.style.width=null;
         });
+
         $('#cases_tab').find('tbody').attr('id','cases_table');
+
         $('#cases_panel').find('.dataTables_controls').find('.dataTables_length').after(
             '<div class="dataTables_goto_page"><label>Page </label><input class="goto-page-number" '
             + 'type="number"><button onclick="changePage(\'cases_tab_wrapper\')">Go</button></div>'
@@ -1129,12 +1132,9 @@ define(['cartutils','filterutils','tippy','jquery', 'base'], function(cartutils,
             updatePromise.reject();
         }
 
-        $('#studies_tab').on('draw.dt', function(){
-            $('#studies_table_head').children('tr').children().each(function(){
-                this.style.width=null;
-                }
-            );
-        })
+        $('#studies_table_head').children('tr').children().each(function(){
+            this.style.width=null;
+        });
 
         $('#studies_tab').children('tbody').attr('id','studies_table');
         $('#studies_tab_wrapper').find('.dataTables_controls').find('.dataTables_length').after(
@@ -1496,10 +1496,8 @@ define(['cartutils','filterutils','tippy','jquery', 'base'], function(cartutils,
             updatePromise.reject();
         }
 
-        $('#series_tab').on('draw.dt', function(){
-            $('#series_table_head').children('tr').children().each(function(){
-                this.style.width=null;
-            });
+        $('#series_table_head').children('tr').children().each(function(){
+            this.style.width=null;
         });
         $('#series_tab').children('tbody').attr('id','series_table');
         $('#series_tab_wrapper').find('.dataTables_controls').find('.dataTables_length').after(

templates/idc/explore_data_core.html

@@ -897,7 +897,6 @@ <h4 class="panel-title">
 
                                 </div>
                                 <div id="projects_panel" class="panel-body">
-
                                     <table id="proj_table" class="table table-striped projects-table" data-table-type="collection">
                                         <thead id="projects_table_head">
                                         <tr>
@@ -921,7 +920,6 @@ <h4 class="panel-title">
                             <div id="cases_panel_container" class="panel panel-default">
                                 <div class="panel-heading clearfix"><h4 class="panel-title">Selected Cases</h4></div>
                                 <div id="cases_panel" class="panel-body">
-
                                     <table id="cases_tab" class="table table-striped cases-table" data-table-type="case">
                                         <thead id="cases_table_head">
                                         <tr>