From 3b463670cecd0d55ff982cbaad39aa5d390bccac Mon Sep 17 00:00:00 2001
From: Felix Niklas <mrflix@gmail.com>
Date: Wed, 28 Oct 2015 11:33:29 +0100
Subject: [PATCH] table resize: fix that some tables have width zero while
 calculating

- fall back minTableWidth
---
 .../_application_controller_table.coffee           | 14 ++++++++++----
 .../javascripts/app/views/generic/table.jst.eco    |  2 +-
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/app/controllers/_application_controller_table.coffee b/app/assets/javascripts/app/controllers/_application_controller_table.coffee
index 5e3f5799f..d1a71fa31 100644
--- a/app/assets/javascripts/app/controllers/_application_controller_table.coffee
+++ b/app/assets/javascripts/app/controllers/_application_controller_table.coffee
@@ -1,6 +1,7 @@
 class App.ControllerTable extends App.Controller
   minColWidth: 40
   baseColWidth: 130
+  minTableWidth: 612
 
   checkBoxColWidth: 40
   radioColWidth: 22
@@ -345,8 +346,13 @@ class App.ControllerTable extends App.Controller
     table
 
   adjustHeaderWidths: (headers) ->
+    availableWidth = @el.width() 
+
+    if availableWidth is 0
+      availableWidth = @minTableWidth
+
     widths = @getHeaderWidths headers
-    difference = widths - @el.width()
+    difference = widths - availableWidth
 
     if difference > 0
       # convert percentages to pixels
@@ -355,12 +361,12 @@ class App.ControllerTable extends App.Controller
 
         if unit is '%'
           percentage = parseInt col.width, 10
-          col.width = percentage / 100 * @el.width() + "px"
+          col.width = percentage / 100 * availableWidth + "px"
 
         return col
 
       widths = @getHeaderWidths headers
-      shrinkBy = Math.ceil (widths - @el.width()) / @getShrinkableHeadersCount(headers)
+      shrinkBy = Math.ceil (widths - availableWidth) / @getShrinkableHeadersCount(headers)
 
       # make all cols evenly smaller
       headers = _.map headers, (col) =>
@@ -370,7 +376,7 @@ class App.ControllerTable extends App.Controller
         return col
 
       # give left-over space from rounding to last column to get to 100%
-      roundingLeftOver = @el.width() - @getHeaderWidths headers
+      roundingLeftOver = availableWidth - @getHeaderWidths headers
       # but only if there is something left over (will get negative when there are too many columns for each column to stay in their min width)
       if roundingLeftOver > 0
         headers[headers.length - 1].width = parseInt(headers[headers.length - 1].width, 10) + roundingLeftOver + "px"
diff --git a/app/assets/javascripts/app/views/generic/table.jst.eco b/app/assets/javascripts/app/views/generic/table.jst.eco
index 1255df7d9..cd3559ed0 100644
--- a/app/assets/javascripts/app/views/generic/table.jst.eco
+++ b/app/assets/javascripts/app/views/generic/table.jst.eco
@@ -15,7 +15,7 @@
     <% end %>
     <% for item, i in @header: %>
       <th class="<%= " #{ item.className }" if item.className %><%= " align-#{ item.align }" if item.align %>" style="width:<%= item.width %>" data-column-key="<%= item.name %>">
-        <div <% if @table_id: %>class="table-column-head js-sort"<% end %>>
+        <div class="table-column-head<%= ' js-sort' if @table_id %>">
           <div class="table-column-title">
           <%- @T( item.display ) %>
           </div>