From 10c6417874129f03f3d2125c5df460865bad2797 Mon Sep 17 00:00:00 2001
From: Luke Shumaker <shumakl@purdue.edu>
Date: Thu, 1 May 2014 17:02:37 -0400
Subject: clean up grep-able issues in forms (mostly using <p> instead of
 <div>)

---
 app/views/users/show.html.erb | 61 +++++++++++++++++++++----------------------
 1 file changed, 30 insertions(+), 31 deletions(-)

(limited to 'app/views/users/show.html.erb')

diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index b247b7d..cd232a3 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -2,56 +2,55 @@
 <h1> <%= @user.user_name %>'s Profile </h1>
 
 <p>
-  <%= image_tag 'http://www.gravatar.com/avatar/' + Digest::MD5.hexdigest(@user.email) + '?s=100&d=identicon' %>
+	<%= image_tag 'http://www.gravatar.com/avatar/' + Digest::MD5.hexdigest(@user.email) + '?s=100&d=identicon' %>
 </p>
 
 <p>
-  <strong>Email:</strong>
-  <%= @user.email %>
-</p>
-
+	<strong>Name:</strong>
+	<%= @user.name %>
 <p>
-  <strong>Preferred name:</strong>
-  <%= @user.user_name %>
-</p>
 
 <p>
-  <strong>Relationship Status:</strong>
-  too single
+	<strong>Alias:</strong>
+	<%= @user.user_name %>
+<p>
+	<strong>Email:</strong>
+	<%= @user.email %>
 </p>
 
-
-<p>
-<% if @user.remote_usernames[0].nil? %>
+<% if @user.remote_usernames.empty? and @user.check_permission(current_user, :edit) %>
 	<%= form_for @user do |f| %>
-		<label>Have a League of Legends Account?
-			<input type=text name="user[remote_usernames][League of Legends]">
-		</label>
-		<%= f.submit "Add Username", :class => 'signup' %>
+		<div class="field">
+			<label for="summoner_name">Have a League of Legends Account?</label>
+			<div class="simple-input-group">
+				<input type="text" id="summoner_name" name="user[remote_usernames][League of Legends]">
+				<%= f.submit "Add Username", :class => 'signup' %>
+			</div>
+		</div>
 	<% end %>
 <% end %>
-</p>
-
 
 <div class="row">
 	<div class="col-md-6">
-	  	<h3> Recent Tournaments Played </h3>
+		<h3>Recent Tournaments Played</h3>
 		<ul>
 			<% @user.tournaments_played.each do |t| %>
 				<li><%= t.name %></li>
 			<% end %>
 		</ul>
-	  </div>
-    <div class="col-md-6">
-    	<h3> Recent Tournaments Hosted </h3>
-    	<% if  @user.tournaments_hosted.count == 0 %> <p> <%= @user.user_name %> has never hosted a tournament </p> <%end %>
-		<ul>
-			<% @user.tournaments_hosted.each do |t| %>
-				<li><%= t.name %></li>
-			<% end %>
-		</ul>
+	</div>
+	<div class="col-md-6">
+		<h3>Recent Tournaments Hosted</h3>
+		<% if  @user.tournaments_hosted.empty? %>
+			<p><%= @user.user_name %> has never hosted a tournament.</p>
+		<% else %>
+			<ul>
+				<% @user.tournaments_hosted.each do |t| %>
+					<li><%= t.name %></li>
+				<% end %>
+			</ul>
+		<% end %>
 	</div>
 </div>
 
-<%= link_to 'Edit', edit_user_path(@user) %> |
-<%= link_to 'All Users', users_path %>
+<%= link_to 'Edit', edit_user_path(@user), role: :button %>
-- 
cgit v1.2.3