6 files changed, 10 insertions, 12 deletions
diff --git a/app/views/admin_request/edit_outgoing.rhtml b/app/views/admin_request/edit_outgoing.rhtml
index 099658e44..d516396bd 100644
--- a/app/views/admin_request/edit_outgoing.rhtml
+++ b/app/views/admin_request/edit_outgoing.rhtml
@@ -16,7 +16,7 @@
   <p>You could also use this to edit a message before resending it, but
   only the edited version will be shown on the public page if you do that.</p>
 
-  <%= submit_tag 'Save' %>
+  <p><%= submit_tag 'Save' %></p>
 <% end %>
 
 <p>
diff --git a/app/views/admin_request/list.rhtml b/app/views/admin_request/list.rhtml
index 9f0ead369..b2874f965 100644
--- a/app/views/admin_request/list.rhtml
+++ b/app/views/admin_request/list.rhtml
@@ -3,10 +3,10 @@
 <h1><%=@title%></h1>
 
 <% form_tag("") do %>
-   </p>
+   <p>
        <%= text_field_tag 'query', params[:query], { :size => 30 } %>
        <%= submit_tag "Search" %> (substring search, titles only)
-   <p>
+   </p>
 <% end %>
 
 <table>
@@ -28,5 +28,3 @@
 
 <%= will_paginate(@info_requests) %>
 
-<br />
-
diff --git a/app/views/admin_request/show.rhtml b/app/views/admin_request/show.rhtml
index 940972187..2556513e6 100644
--- a/app/views/admin_request/show.rhtml
+++ b/app/views/admin_request/show.rhtml
@@ -37,8 +37,10 @@
   <% end %>
   <td>
   <% form_tag '../resend' do %>
-      <%= hidden_field_tag 'outgoing_message_id', outgoing_message.id %>
-      <%= submit_tag "Resend" %>
+      <div>
+          <%= hidden_field_tag 'outgoing_message_id', outgoing_message.id %>
+          <%= submit_tag "Resend" %>
+      </div>
   <% end %>
   <%= link_to "Edit", '../edit_outgoing/' + outgoing_message.id.to_s %>
   </td>
diff --git a/app/views/admin_user/list.rhtml b/app/views/admin_user/list.rhtml
index 5a575bce2..b84b81b0a 100644
--- a/app/views/admin_user/list.rhtml
+++ b/app/views/admin_user/list.rhtml
@@ -21,7 +21,7 @@
   <tr class="<%= cycle('odd', 'even') %>">
     <td><%= user.id.to_s %></td>
     <td><%= link_to h(user.name), 'show/' + user.id.to_s %></td>
-    <td><a href="mailto:<%=h user.email %>"><%=h user.email%></td>
+    <td><a href="mailto:<%=h user.email %>"><%=h user.email%></a></td>
   <% for column in ['created_at', 'updated_at', 'email_confirmed'] %>
     <td><%=h user.send(column) %></td>
   <% end %>
@@ -31,5 +31,3 @@
 
 <%= will_paginate(@admin_users) %>
 
-<br />
-
diff --git a/app/views/help/about.rhtml b/app/views/help/about.rhtml
index 3eef0b58a..453d8ab79 100644
--- a/app/views/help/about.rhtml
+++ b/app/views/help/about.rhtml
@@ -23,7 +23,7 @@ to the public authority. Any response they make is automatically published on th
 website for you and anyone else to find and read.
 </dd>
 
-<dt id="whybother_me">Why would I bother to do this?</a>
+<dt id="whybother_me">Why would I bother to do this?</dt>
 <dd>You pay taxes, and then government does things with the money. All sorts of
 things that affect your life, from healthcare through to national defence. Some
 it does badly, some it does well. The more we find out about how government
diff --git a/app/views/help/contact.rhtml b/app/views/help/contact.rhtml
index e8ae88793..c197dc5bf 100644
--- a/app/views/help/contact.rhtml
+++ b/app/views/help/contact.rhtml
@@ -51,7 +51,7 @@
     </p>
 
     <p>
-        <label class="form_label" for="outgoing_message_body">Message:</label> 
+        <label class="form_label" for="contact_message">Message:</label> 
         <%= f.text_area :message, :rows => 5, :cols => 50 %>
     </p>