Changed AR error messages style (same as flash errors).

Fixed flash styles for IE6. git-svn-id: http://redmine.rubyforge.org/svn/trunk@610 e93f8b46-1217-0410-a6f0-8f06a7374b81

Changed AR error messages style (same as flash errors).
Fixed flash styles for IE6. git-svn-id: http://redmine.rubyforge.org/svn/trunk@610 e93f8b46-1217-0410-a6f0-8f06a7374b81
e9ba8a44 · Jean-Philippe Lang · 58ea44a4 · e9ba8a44 · e9ba8a44 · 58ea44a4
Commit e9ba8a44 authored Aug 12, 2007 by Jean-Philippe Lang
5 changed files
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -211,7 +211,7 @@ module ApplicationHelper
      end      
      content_tag("div",
        content_tag(
-          options[:header_tag] || "h2", lwr(:gui_validation_error, full_messages.length) + " :"
+          options[:header_tag] || "span", lwr(:gui_validation_error, full_messages.length) + ":"
        ) +
        content_tag("ul", full_messages.collect { |msg| content_tag("li", msg) }),
        "id" => options[:id] || "errorExplanation", "class" => options[:class] || "errorExplanation"

--- a/app/views/layouts/base.rhtml
+++ b/app/views/layouts/base.rhtml
@@ -138,8 +138,10 @@
 	</div>
 	
 	<div id="content">
-		<%= content_tag('div', flash[:error], :id => 'flash', :class => 'error') if flash[:error] %>
-		<%= content_tag('div', flash[:notice], :id => 'flash', :class => 'notice') if flash[:notice] %>
+	    <div id="flash">
+		  <%= content_tag('div', flash[:error], :class => 'error') if flash[:error] %>
+		  <%= content_tag('div', flash[:notice], :class => 'notice') if flash[:notice] %>
+		</div>
 		<%= yield %>
 	</div>
 	

--- a/public/images/alert.png
+++ b/public/images/alert.png
--- a/public/images/false.png
+++ b/public/images/false.png
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -315,36 +315,6 @@ table.list tbody th {
    color: #777;
 }

-/********** Validation error messages *************/
-#errorExplanation {
-  width: 400px;
-  border: 0;
-  padding: 7px;
-  padding-bottom: 3px;
-  margin-bottom: 0px;
-}
-
-#errorExplanation h2 {
-  text-align: left;
-  font-weight: bold;
-  padding: 5px 5px 10px 26px;
-  font-size: 1em;
-  margin: -7px;
-  background: url(../images/alert.png) no-repeat 6px 6px;
-}
-
-#errorExplanation p {
-  color: #333;
-  margin-bottom: 0;
-  padding: 5px;
-}
-
-#errorExplanation ul li {
-  font-size: 1em;
-  list-style: none;
-  margin-left: -16px;
-}
-
 /*==========  Drop down menu  ==============*/
 div.menu {
  background-color: #FFFFFF;
@@ -696,24 +666,27 @@ padding-left: 26px;
 vertical-align: bottom;
 }

-/***** Flash messages ****/
-#flash {
+/***** Flash & error messages ****/
+#flash div, #errorExplanation {
    padding: 4px 4px 4px 30px;
    margin-bottom: 16px;
-	font-size: 1.2em;
+	font-size: 1.1em;
 	border: 2px solid;
 }

-#flash.error {
-    background: url(../images/false.png) 8px 50% no-repeat;
+#flash div.error, #errorExplanation {
+    background: url(../images/false.png) 8px 5px no-repeat;
 	background-color: #ffe3e3;
 	border-color: #dd0000;
 	color: #550000;
 }

-#flash.notice {
-    background: url(../images/true.png) 8px 50% no-repeat;
+#flash div.notice {
+    background: url(../images/true.png) 8px 5px no-repeat;
 	background-color: #dfffdf;
 	border-color: #9fcf9f;
 	color: #005f00;
 }
+
+#errorExplanation ul { margin-bottom: 0px; }
+#errorExplanation ul li { list-style: none; margin-left: -16px;}