From c51b214ef473aaa72d410d92a3c67b2e68af70cc Mon Sep 17 00:00:00 2001
From: Henriette Dinger <h.dinger@finn.de>
Date: Fri, 4 Mar 2016 14:09:58 +0100
Subject: [PATCH] Use fieldset classes for correct styling

---
 app/assets/stylesheets/costs/costs_legacy.css |  4 ----
 app/views/cost_objects/_form.html.erb         | 22 +++++++++++++------
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/app/assets/stylesheets/costs/costs_legacy.css b/app/assets/stylesheets/costs/costs_legacy.css
index 6604512447..439d5e96c6 100644
--- a/app/assets/stylesheets/costs/costs_legacy.css
+++ b/app/assets/stylesheets/costs/costs_legacy.css
@@ -98,10 +98,6 @@ td.comment input {
   cursor: pointer;
 }
 
-.budget-table--fieldset{
-  margin: 8px 2px;
-}
-
 .cost_entry td{
   vertical-align: top;
   padding: 6px 6px;
diff --git a/app/views/cost_objects/_form.html.erb b/app/views/cost_objects/_form.html.erb
index 808557773c..9bb5676a0c 100644
--- a/app/views/cost_objects/_form.html.erb
+++ b/app/views/cost_objects/_form.html.erb
@@ -44,8 +44,8 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
   //]]>
 </script>
 
-<fieldset id="material_budget_items_fieldset" class="budget-table--fieldset">
-  <legend><%= VariableCostObject.human_attribute_name(:material_budget) %></legend>
+<fieldset id="material_budget_items_fieldset" class="form--fieldset -collapsible">
+  <legend class="form--fieldset-legend" onclick="toggleFieldset(this);"><%= VariableCostObject.human_attribute_name(:material_budget) %></legend>
     <div class="generic-table--container">
       <div class="generic-table--results-container">
         <table interactive-table class="generic-table" id="material_budget_items">
@@ -87,6 +87,15 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
                 </div>
               </div>
             </th>
+            <th>
+              <div class="generic-table--sort-header-outer">
+                <div class="generic-table--sort-header">
+                  <span>
+                    <%= MaterialBudgetItem.human_attribute_name(:description) %>
+                  </span>
+                </div>
+              </div>
+            </th>
             <% if User.current.allowed_to?(:view_cost_rates, @project)%>
               <th class="currency" id="material_budget_items_price">
                 <div class="generic-table--sort-header-outer">
@@ -98,7 +107,6 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
                 </div>
               </th>
             <%end%>
-            <th></th>
             <th class="-short"></th>
           </tr>
         </thead>
@@ -112,12 +120,12 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
     </div>
   </div>
   <div class="generic-table--action-buttons">
-    <%= link_to_function l(:button_add_budget_item), "materialBudgetItemsForm.add()", {class: "button -with-icon icon-context icon-add"} %>
+    <%= link_to_function l(:button_add_budget_item), "materialBudgetItemsForm.add()", {class: "button -alt-highlight -with-icon icon-context icon-add"} %>
   </div>
 </fieldset>
 
-<fieldset id="labor_budget_items_fieldset" class="budget-table--fieldset">
-  <legend><%= VariableCostObject.human_attribute_name(:labor_budget) %></legend>
+<fieldset id="labor_budget_items_fieldset" class="form--fieldset -collapsible">
+  <legend class="form--fieldset-legend" onclick="toggleFieldset(this);"><%= VariableCostObject.human_attribute_name(:labor_budget) %></legend>
   <div class="generic-table--container">
     <div class="generic-table--results-container">
       <table interactive-table class="generic-table" id="labor_budget_items">
@@ -183,7 +191,7 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
     </div>
   </div>
   <div class="generic-table--action-buttons">
-    <%= link_to_function l(:button_add_budget_item), "laborBudgetItemsForm.add()", {class: "button -with-icon icon-context icon-add"} %>
+    <%= link_to_function l(:button_add_budget_item), "laborBudgetItemsForm.add()", {class: "button -alt-highlight -with-icon icon-context icon-add"} %>
   </div>
 </fieldset>