From 90d2aecceaf115946c2d15bef76eaf3612605076 Mon Sep 17 00:00:00 2001
From: Eric Davis <edavis@littlestreamsoftware.com>
Date: Mon, 9 Aug 2010 11:07:55 -0700
Subject: [PATCH] [#4178] Added styles for the Deliverable form.

---
 app/views/deliverables/_form.html.erb    | 69 ++++++++++++++++++------
 assets/stylesheets/redmine_contracts.css |  2 -
 config/locales/en.yml                    |  1 +
 3 files changed, 54 insertions(+), 18 deletions(-)

diff --git a/app/views/deliverables/_form.html.erb b/app/views/deliverables/_form.html.erb
index d0af264..d47f435 100644
--- a/app/views/deliverables/_form.html.erb
+++ b/app/views/deliverables/_form.html.erb
@@ -1,4 +1,5 @@
-<% form.inputs do %>
+<div class="box tabular">
+<% form.inputs :name => l(:text_deliverable_details_legend) do %>
   <%= form.input :title, :required => true %>
   <% if resource.new_record? %>
     <%= form.input(:type, :required => true, :as => :select, :collection => [["Fixed", "FixedDeliverable"],["Hourly", "HourlyDeliverable"]], :include_blank => false, :input_html => {:class => 'type'}) %>
@@ -20,30 +21,66 @@
     <%= check_box resource.class.to_s.underscore, 'warranty_sign_off' %>
   </li>
   <% end %>
+<% end %>
 
-  <% form.inputs :name => l(:text_deliverable_finances), :id => 'deliverable-finances' do %>
-    <% form.semantic_fields_for :labor_budgets do |labor_budget| %>
-      <% labor_budget.inputs :name => l(:field_labor), :id => 'deliverable-labor' do %>
-        <%= labor_budget.input :hours, :label => l(:text_short_hours), :input_html => {:size => 10} %>
-        <%= labor_budget.input :budget, :label => l(:text_dollar_sign), :input_html => {:size => 10} %>
-      <% end %>
-    <% end %>
+<% form.inputs :name => l(:text_deliverable_finances), :id => 'deliverable-finances' do %>
+  <% form.fields_for :labor_budgets do |labor_budget| %>
+    <li class="numeric optional">
 
-    <% form.semantic_fields_for :overhead_budgets do |overhead_budget| %>
-      <% overhead_budget.inputs :name => l(:field_overhead), :id => 'deliverable-overhead' do %>
-        <%= overhead_budget.input :hours, :label => l(:text_short_hours), :input_html => {:size => 10} %>
-        <%= overhead_budget.input :budget, :label => l(:text_dollar_sign), :input_html => {:size => 10} %>
-      <% end %>
-    <% end %>
+    <%= content_tag(:label, l(:field_labor)) %>
+    <table class="deliverable_finance_table">
+		  <tr>
+			  <td></td>
+			  <td>
+				  <p class="inline-hints"><%= l(:text_short_hours) %></p>
+          <%= labor_budget.text_field(:hours, :size => 10) %>
+			  </td>
+			  <td>
+				  <p class="inline-hints"><%= l(:text_dollar_sign) %></p>
+          <%= labor_budget.text_field(:budget, :size => 10) %>
+			  </td>
+			  <td></td>
+		  </tr>
+	  </table>
+
+    </li>
+  <% end %>
+
+  <% form.fields_for :overhead_budgets do |overhead_budget| %>
+    <li class="numeric optional">
+
+    <%= content_tag(:label, l(:field_overhead)) %>
+    <table class="deliverable_finance_table">
+		  <tr>
+			  <td></td>
+			  <td>
+				  <p class="inline-hints"><%= l(:text_short_hours) %></p>
+          <%= overhead_budget.text_field(:hours, :size => 10) %>
+			  </td>
+			  <td>
+				  <p class="inline-hints"><%= l(:text_dollar_sign) %></p>
+          <%= overhead_budget.text_field(:budget, :size => 10) %>
+			  </td>
+			  <td></td>
+		  </tr>
+	  </table>
+
+    </li>
   <% end %>
 
   <%= form.input :total, :input_html => {:size => 10}, :wrapper_html => {:class => 'deliverable_total_input'}, :hint => l(:text_dollar_sign) %>
 
 <% end %>
 
+</div>
+
 <% form.buttons do %>
-  <%= form.commit_button :label => l(:button_save) %>
-  <%= link_to(l(:button_cancel), cancel_path) %>
+  <ol>
+    <li class="commit">
+      <%= submit_tag(l(:button_save)) %>
+      <%= link_to(l(:button_cancel), cancel_path) %>
+    </li>
+  </ol>
 <% end %>
 
 <%= wikitoolbar_for resource.to_underscore + '_notes' %>
diff --git a/assets/stylesheets/redmine_contracts.css b/assets/stylesheets/redmine_contracts.css
index a79dde8..0a2d3cb 100644
--- a/assets/stylesheets/redmine_contracts.css
+++ b/assets/stylesheets/redmine_contracts.css
@@ -29,8 +29,6 @@ html>body .tabular li {overflow:hidden;}
 a.contract-delete {color: red; }
 
 /* Positioning */
-#deliverable-finances ol li { display: inline; }
-#deliverable-finances ol li label { float: none; margin-left: 0; }
 
 #deliverables td.labor, #deliverables td.overhead, #deliverables td.fixed { text-align: center; }
 #deliverables td.labor div, #deliverables td.overhead div, #deliverables td.fixed div { text-align: right; display: inline; padding: 0 5px; width: 50%; }
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 4f86183..ccd70b2 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -58,5 +58,6 @@ en:
   text_general_legend: "General"
   text_budget_legend: "Budget"
   text_account_legend: "Account Management"
+  text_deliverable_details_legend: "Deliverable Details"
   text_save_contract: "Save Contract"