From d7a0fffe1aab9de4b18db88c409fcd5dc486ab5c Mon Sep 17 00:00:00 2001
From: Luke Shumaker <shumakl@purdue.edu>
Date: Thu, 1 May 2014 16:19:34 -0400
Subject: clean up forms

---
 app/views/tournaments/_form.html.erb | 43 ++++++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 17 deletions(-)

(limited to 'app/views/tournaments/_form.html.erb')

diff --git a/app/views/tournaments/_form.html.erb b/app/views/tournaments/_form.html.erb
index fe2a5c6..93badeb 100644
--- a/app/views/tournaments/_form.html.erb
+++ b/app/views/tournaments/_form.html.erb
@@ -1,12 +1,20 @@
 <%= render "common/error_messages", :target => @tournament %>
 <fieldset>
-	<legend>Game Type</legend>
+	<legend>Basic information</legend>
 	<%= form_for(@tournament, url: new_tournament_path, method: "get") do |f| %>
-		<p>
+		<div class="field">
 			<%= f.label :game_id, "Select a Game Type" %>
 			<%= f.select(:game_id, Game.all.map{|game| [game.name, game.id]}) %>
-			<%= f.submit("Select") %>
-		</p>
+		</div>
+		<div class="field">
+		    <label for="num_stages">Number of Tournament Stages</label>
+		    <input type="number" id="num_stages" name="num_stages" min="1" value="<%= params[:num_stages].to_i %>">
+		</div>
+		<% if @tournament.game %>
+			<%= f.submit("Update (Will reset the rest of the form)", class: "btn-danger") %>
+		<% else %>
+			<%= f.submit("Submit") %>
+		<% end %>
 	<% end %>
 
 </fieldset>
@@ -16,10 +24,10 @@
 			<legend>Attributes</legend>
 
 			<%= f.hidden_field(:game_id) %>
-			<p>
+			<div class="field">
 				<%= f.label :name %>
 				<%= f.text_field :name %>
-			</p>
+			</div>
 
 			<table>
 				<tbody>
@@ -41,17 +49,17 @@
 				</tbody>
 			</table>
 
-			<p>
+			<div class="field">
 				<%= f.label :scoring_method, :scoring_method.to_s.titleize %>
 				<%= f.select(:scoring_method, @tournament.scoring_methods.map{|method| [method.humanize.titleize, method]}) %>
-			</p>
+			</div>
 		</fieldset>
 
 		<fieldset>
 			<legend>Settings</legend>
 			<%= f.fields_for :settings do |setting_fields| %>
 				<% @tournament.tournament_settings.each do |setting| %>
-					<p>
+					<div class="field">
 						<%= setting_fields.label setting.name, setting.name.to_s.titleize %>
 						<br>
 						<% case setting.vartype %>
@@ -77,7 +85,7 @@
 						<% when 5 %>
 							<%= setting_fields.select( setting.name, setting.type_opt.split(',').collect {|opt| opt.humanize.titleize} ) %>
 						<% end %>
-					</p>
+					</div>
 				<% end %>
 			<% end %>
 		</fieldset>
@@ -85,17 +93,18 @@
 		<%= f.fields_for :stages do |stages_fields| %>
 			<fieldset>
 				<legend>Stages</legend>
-				<label for="num_stages">Number of Tournament Stages</label>
-				<input type="number" id="num_stages" name="num_stages" min="1" value="<%= params[:num_stages].to_i %>">
-				<%# stage_fields.submit("Set Stages") %>
 				<% for i in 1..(params[:num_stages].to_i) do %>
 					<%= stages_fields.fields_for i.to_s do |stage_fields| %>
 					<fieldset>
 						<legend>Stage <%= i %></legend>
-						<%= stage_fields.label :scheduling_method, :scheduling_method.to_s.titleize %>
-						<%= stage_fields.select(:scheduling_method, @tournament.scheduling_methods.map{|method| [method.humanize.titleize, method]}) %>
-						<%= stage_fields.label :seeding_method, :seeding_method.to_s.titleize %>
-						<%= stage_fields.select(:seeding_method, @tournament.seeding_methods.map{|method| [method.humanize.titleize, method]}) %>
+						<div class="field">
+						    <%= stage_fields.label :scheduling_method, :scheduling_method.to_s.titleize %>
+						    <%= stage_fields.select(:scheduling_method, @tournament.scheduling_methods.map{|method| [method.humanize.titleize, method]}) %>
+						</div>
+						<div class="field">
+						    <%= stage_fields.label :seeding_method, :seeding_method.to_s.titleize %>
+						    <%= stage_fields.select(:seeding_method, @tournament.seeding_methods.map{|method| [method.humanize.titleize, method]}) %>
+						</div>
 					</fieldset>
 					<% end %>
 				<% end %>
-- 
cgit v1.2.3