From c19e3e0627d6727d66941973a1b2cc4ceb3c1b80 Mon Sep 17 00:00:00 2001
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sun, 3 Jul 2011 12:21:37 +0000
Subject: [PATCH] Replaced h3+div with fieldsets on my account and user form.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@6165 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 app/views/my/account.rhtml  | 48 ++++++++++++------------
 app/views/users/_form.rhtml | 74 ++++++++++++++++++-------------------
 2 files changed, 61 insertions(+), 61 deletions(-)

diff --git a/app/views/my/account.rhtml b/app/views/my/account.rhtml
index 139ca6b1a..c1579cde6 100644
--- a/app/views/my/account.rhtml
+++ b/app/views/my/account.rhtml
@@ -2,43 +2,43 @@
 <%= link_to(l(:button_change_password), {:action => 'password'}, :class => 'icon icon-passwd') if @user.change_password_allowed? %>
 <%= call_hook(:view_my_account_contextual, :user => @user)%>
 </div>
+
 <h2><%=l(:label_my_account)%></h2>
-<%= error_messages_for 'user' %>
 
 <% form_for :user, @user, :url => { :action => "account" }, 
                           :builder => TabularFormBuilder, 
                           :lang => current_language,
                           :html => { :id => 'my_account_form' } do |f| %>
 <div class="splitcontentleft">
-<h3><%=l(:label_information_plural)%></h3>
-<div class="box tabular">
-<p><%= f.text_field :firstname, :required => true %></p>
-<p><%= f.text_field :lastname, :required => true %></p>
-<p><%= f.text_field :mail, :required => true %></p>
-<p><%= f.select :language, lang_options_for_select %></p>
-<% if Setting.openid? %>
-<p><%= f.text_field :identity_url  %></p>
-<% end %>
-
-<% @user.custom_field_values.select(&:editable?).each do |value| %>
-	<p><%= custom_field_tag_with_label :user, value %></p>
-<% end %>
-<%= call_hook(:view_my_account, :user => @user, :form => f) %>
-</div>
+<fieldset class="box tabular">
+	<legend><%=l(:label_information_plural)%></legend>
+	<p><%= f.text_field :firstname, :required => true %></p>
+	<p><%= f.text_field :lastname, :required => true %></p>
+	<p><%= f.text_field :mail, :required => true %></p>
+	<p><%= f.select :language, lang_options_for_select %></p>
+	<% if Setting.openid? %>
+	<p><%= f.text_field :identity_url  %></p>
+	<% end %>
+	
+	<% @user.custom_field_values.select(&:editable?).each do |value| %>
+		<p><%= custom_field_tag_with_label :user, value %></p>
+	<% end %>
+	<%= call_hook(:view_my_account, :user => @user, :form => f) %>
+</fieldset>
 
 <%= submit_tag l(:button_save) %>
 </div>
 
 <div class="splitcontentright">
-<h3><%=l(:field_mail_notification)%></h3>
-<div class="box">
-<%= render :partial => 'users/mail_notifications' %>
-</div>
+<fieldset class="box">
+	<legend><%=l(:field_mail_notification)%></legend>
+	<%= render :partial => 'users/mail_notifications' %>
+</fieldset>
 
-<h3><%=l(:label_preferences)%></h3>
-<div class="box tabular">
-<%= render :partial => 'users/preferences' %>
-</div>
+<fieldset class="box tabular">
+	<legend><%=l(:label_preferences)%></legend>
+	<%= render :partial => 'users/preferences' %>
+</fieldset>
 
 </div>
 <% end %>
diff --git a/app/views/users/_form.rhtml b/app/views/users/_form.rhtml
index f23635693..4b145bc3d 100644
--- a/app/views/users/_form.rhtml
+++ b/app/views/users/_form.rhtml
@@ -3,48 +3,48 @@
 <div id="user_form">
 <!--[form:user]-->
 <div class="splitcontentleft">
-<h3><%=l(:label_information_plural)%></h3>
-<div class="box tabular">
-<p><%= f.text_field :login, :required => true, :size => 25  %></p>
-<p><%= f.text_field :firstname, :required => true %></p>
-<p><%= f.text_field :lastname, :required => true %></p>
-<p><%= f.text_field :mail, :required => true %></p>
-<p><%= f.select :language, lang_options_for_select %></p>
-<% if Setting.openid? %>
-<p><%= f.text_field :identity_url  %></p>
-<% end %>
+<fieldset class="box tabular">
+	<legend><%=l(:label_information_plural)%></legend>
+	<p><%= f.text_field :login, :required => true, :size => 25  %></p>
+	<p><%= f.text_field :firstname, :required => true %></p>
+	<p><%= f.text_field :lastname, :required => true %></p>
+	<p><%= f.text_field :mail, :required => true %></p>
+	<p><%= f.select :language, lang_options_for_select %></p>
+	<% if Setting.openid? %>
+	<p><%= f.text_field :identity_url  %></p>
+	<% end %>
+	
+	<% @user.custom_field_values.each do |value| %>
+		<p><%= custom_field_tag_with_label :user, value %></p>
+	<% end %>
+	
+	<p><%= f.check_box :admin, :disabled => (@user == User.current) %></p>
+	<%= call_hook(:view_users_form, :user => @user, :form => f) %>
+</fieldset>
 
-<% @user.custom_field_values.each do |value| %>
-	<p><%= custom_field_tag_with_label :user, value %></p>
-<% end %>
-
-<p><%= f.check_box :admin, :disabled => (@user == User.current) %></p>
-<%= call_hook(:view_users_form, :user => @user, :form => f) %>
-</div>
-
-<h3><%=l(:label_authentication)%></h3>
-<div class="box tabular">
-<% unless @auth_sources.empty? %>
-<p><%= f.select :auth_source_id, ([[l(:label_internal), ""]] + @auth_sources.collect { |a| [a.name, a.id] }), {}, :onchange => "if (this.value=='') {Element.show('password_fields');} else {Element.hide('password_fields');}" %></p>
-<% end %>
-<div id="password_fields" style="<%= 'display:none;' if @user.auth_source %>">
-<p><%= f.password_field :password, :required => true, :size => 25  %><br />
-<em><%= l(:text_caracters_minimum, :count => Setting.password_min_length) %></em></p>
-<p><%= f.password_field :password_confirmation, :required => true, :size => 25  %></p>
-</div>
-</div>
+<fieldset class="box tabular">
+	<legend><%=l(:label_authentication)%></legend>
+	<% unless @auth_sources.empty? %>
+	<p><%= f.select :auth_source_id, ([[l(:label_internal), ""]] + @auth_sources.collect { |a| [a.name, a.id] }), {}, :onchange => "if (this.value=='') {Element.show('password_fields');} else {Element.hide('password_fields');}" %></p>
+	<% end %>
+	<div id="password_fields" style="<%= 'display:none;' if @user.auth_source %>">
+	<p><%= f.password_field :password, :required => true, :size => 25  %><br />
+	<em><%= l(:text_caracters_minimum, :count => Setting.password_min_length) %></em></p>
+	<p><%= f.password_field :password_confirmation, :required => true, :size => 25  %></p>
+	</div>
+</fieldset>
 </div>
 
 <div class="splitcontentright">
-<h3><%=l(:field_mail_notification)%></h3>
-<div class="box">
-<%= render :partial => 'users/mail_notifications' %>
-</div>
+<fieldset class="box">
+	<legend><%=l(:field_mail_notification)%></legend>
+	<%= render :partial => 'users/mail_notifications' %>
+</fieldset>
 
-<h3><%=l(:label_preferences)%></h3>
-<div class="box tabular">
-<%= render :partial => 'users/preferences' %>
-</div>
+<fieldset class="box tabular">
+	<legend><%=l(:label_preferences)%></legend>
+	<%= render :partial => 'users/preferences' %>
+</fieldset>
 </div>
 </div>
 <div style="clear:left;"></div>