Inline validation errors on sign up page

[rails.git] / app / views / user / new.html.erb

diff --git a/app/views/user/new.html.erb b/app/views/user/new.html.erb
index 7324ddd29773a8edb0c585afa122511245d1a289..6c89a7830d5b8f6461822c26631981433d2a166c 100644 (file)
--- a/app/views/user/new.html.erb
+++ b/app/views/user/new.html.erb
@@ -1,102 +1,100 @@
 <% content_for :heading do %>
-  <h1><%= t 'user.new.heading' %></h1>
+  <h1><%= t 'user.new.title' %></h1>
+  <div class='header-illustration new-user-main'></div>
+  <div class='header-illustration new-user-arm'></div>
 <% end %>
 
-<p><%= t 'user.new.fill_form' %></p>
-
-<%= error_messages_for 'user' %>
-
-<%= form_for :user, :url => { :action => 'create' } do %>
+<%= form_for :user, :url => { :action => 'create' }, :html => { :class => 'signup-form-wrapper standard-form fillL col6 pad2' } do %>
   <%= hidden_field_tag('referer', h(@referer)) unless @referer.nil? %>
 
-  <div id="signupForm" class="standard-form">
-
-    <fieldset>
-      <div class="form-row">
-        <label for="email" class="standard-label">
-          <%= t 'user.new.email address' %>
-        </label>
-        <%= email_field(:user, :email, { :tabindex => 1 }) %>
-      </div>
-      <div class="form-row">
-        <label for="email_confirmation" class="standard-label">
-          <%= t 'user.new.confirm email address' %>
-        </label>
-        <%= email_field(:user, :email_confirmation, { :tabindex => 2 }) %>
-      </div>
+  <fieldset>
+    <div class="form-row">
+      <label for="email" class="standard-label">
+        <%= t 'user.new.email address' %>
+      </label>
+      <%= email_field(:user, :email, { :tabindex => 1 }) %>
+      <%= error_message_on(:user, :email) %>
+    </div>
+    <div class="form-row">
+      <label for="email_confirmation" class="standard-label">
+        <%= t 'user.new.confirm email address' %>
+      </label>
+      <%= email_field(:user, :email_confirmation, { :tabindex => 2 }) %>
+      <%= error_message_on(:user, :email_confirmation) %>
+    </div>
     <span class="form-help deemphasize"><%= raw(t 'user.new.not displayed publicly') %></span>
-    </fieldset>
-
-    <fieldset>
-      <div class="form-row">
-        <label for="display_name" class="standard-label">
-          <%= t 'user.new.display name' %>
-        </label>
-        <%= text_field(:user, :display_name, { :tabindex => 3 }) %>
-      </div>
-      <span class="form-help deemphasize"><%= t 'user.new.display name description' %></span>
-    </fieldset>
-
-    <fieldset class="form-divider" id="openid_field">
-      <div class="form-row">
-        <label for="openid_url" class="standard-label">
-          <%= raw t 'user.new.openid', :logo => openid_logo %>
-        </label>
-        <%= url_field(:user, :openid_url, { :id => "openid_url", :tabindex => 4, :class => "openid_url" }) %>
-      </div>
-      <span id="openid_note" class="deemphasize">
-        <%= t 'user.new.openid no password' %>
-      </span>
-    </fieldset>
-
-    <fieldset>
-      <div class="form-row">
-        <label for='user[pass_crypt]' class="standard-label">
-          <%= t 'user.new.password' %>
-        </label>
-        <%= password_field(:user, :pass_crypt, {:tabindex => 5 }) %>
-      </div>
-      <div class="form-row">
-        <label class="standard-label">
-          <%= t 'user.new.confirm password' %>
-        </label>
-        <%= password_field(:user, :pass_crypt_confirmation, { :tabindex => 6 }) %>
-      </div>
-    </fieldset>
-
-    <div id="openid_prompt" class="form-row">
-      <%= link_to raw(t("user.new.use openid", :logo => openid_logo)), "#", :id => "openid_enable" %>
+  </fieldset>
+
+  <fieldset>
+    <div class="form-row">
+      <label for="display_name" class="standard-label">
+        <%= t 'user.new.display name' %>
+      </label>
+      <%= text_field(:user, :display_name, { :tabindex => 3 }) %>
+      <%= error_message_on(:user, :display_name) %>
     </div>
+    <span class="form-help deemphasize"><%= t 'user.new.display name description' %></span>
+  </fieldset>
+
+  <fieldset class="form-divider" id="openid_field">
+    <div class="form-row">
+      <label for="openid_url" class="standard-label">
+        <%= raw t 'user.new.openid', :logo => openid_logo %>
+      </label>
+      <%= url_field(:user, :openid_url, { :id => "openid_url", :tabindex => 4, :class => "openid_url" }) %>
+      <%= error_message_on(:user, :openid_url) %>
+    </div>
+    <span class="form-help deemphasize"><%= t 'user.new.openid no password' %></span>
+  </fieldset>
+
+  <fieldset>
+    <div class="form-row">
+      <label for='user[pass_crypt]' class="standard-label">
+        <%= t 'user.new.password' %>
+      </label>
+      <%= password_field(:user, :pass_crypt, {:tabindex => 5 }) %>
+      <%= error_message_on(:user, :pass_crypt) %>
+    </div>
+    <div class="form-row">
+      <label class="standard-label">
+        <%= t 'user.new.confirm password' %>
+      </label>
+      <%= password_field(:user, :pass_crypt_confirmation, { :tabindex => 6 }) %>
+      <%= error_message_on(:user, :pass_crypt_confirmation) %>
+    </div>
+  </fieldset>
 
-    <%= submit_tag t('user.new.continue'), :tabindex => 6 %>
+  <div id="openid_prompt" class="form-row">
+    <%= link_to raw(t("user.new.use openid", :logo => openid_logo)), "#", :id => "openid_enable" %>
   </div>
+
+  <%= submit_tag t('user.new.continue'), :tabindex => 6 %>
 <% end %>
 
+<div class='col6 deemphasize pad2'>
+  <h4><%= t 'user.new.about.header' %></h4>
+  <%= t 'user.new.about.html' %>
+</div>
+
 <script type="text/javascript">
 function enableOpenID()
 {
   $("#openid_prompt").hide();
-
   $("#openid_field").show();
-  $("#openid_note").show();
-
   $("#openid_url").prop("disabled", false);
 }
 
 function disableOpenID()
 {
   $("#openid_prompt").show();
-
   $("#openid_field").hide();
-  $("#openid_note").hide();
-
   $("#openid_url").prop("disabled", true);
 }
 
 $(document).ready(function () {
   $("#openid_enable").click(enableOpenID);
 
-  <% if params[:openid] or (@user and @user.openid_url and not @user.openid_url.empty?) -%>
+  <% if params[:openid] or (@user and @user.openid_url.present?) -%>
   enableOpenID();
   <% else -%>
   disableOpenID();