From ba7bacead84bb67e1b049b281b9e13a5c8af8b1a Mon Sep 17 00:00:00 2001
From: Carsten Hufe <carsten.hufe@devproof.org>
Date: Sun, 27 Mar 2011 22:22:17 +0200
Subject: [PATCH] better comments form

---
 .../portal/module/comment/css/comment.css       | 17 +++++++++++++++--
 .../module/comment/panel/CommentPanel.java      |  7 ++++++-
 .../portal/core/module/common/css/default.css   |  1 -
 3 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/module-comment/src/main/java/org/devproof/portal/module/comment/css/comment.css b/module-comment/src/main/java/org/devproof/portal/module/comment/css/comment.css
index 0390bc07..ee663869 100644
--- a/module-comment/src/main/java/org/devproof/portal/module/comment/css/comment.css
+++ b/module-comment/src/main/java/org/devproof/portal/module/comment/css/comment.css
@@ -85,13 +85,13 @@
 }
 
 .commentFormL {
-	width:170px;
+	width:200px;
 	margin-right:10px;
 	float:left;
 	margin-top:20px;
 }
 .commentFormR {
-	width:362px;
+	width:400px;
 	margin-right:10px;
 	float:left;
 	margin-top:20px;
@@ -136,4 +136,17 @@ input.commentSubmit {
 	font-weight:normal;
 	float:right;
 	cursor:pointer;
+    margin-right: 38px;
+}
+
+input.error,
+select.error,
+textarea.error {
+    background-color: #FFC700;
+}
+
+select.commentInputArea:focus,
+input.commentInputArea:focus,
+textarea.commenttextarea:focus{
+    background-color:#FFEBAD;
 }
\ No newline at end of file
diff --git a/module-comment/src/main/java/org/devproof/portal/module/comment/panel/CommentPanel.java b/module-comment/src/main/java/org/devproof/portal/module/comment/panel/CommentPanel.java
index 622297c8..35ca1e90 100644
--- a/module-comment/src/main/java/org/devproof/portal/module/comment/panel/CommentPanel.java
+++ b/module-comment/src/main/java/org/devproof/portal/module/comment/panel/CommentPanel.java
@@ -42,6 +42,8 @@
 import org.apache.wicket.validation.validator.EmailAddressValidator;
 import org.apache.wicket.validation.validator.StringValidator;
 import org.devproof.portal.core.app.PortalSession;
+import org.devproof.portal.core.module.common.component.PortalFeedbackPanel;
+import org.devproof.portal.core.module.common.component.ValidationDisplayBehaviour;
 import org.devproof.portal.core.module.common.dataprovider.QueryDataProvider;
 import org.devproof.portal.core.module.common.panel.BubblePanel;
 import org.devproof.portal.core.module.common.panel.captcha.CaptchaAjaxButton;
@@ -120,6 +122,7 @@ private WebMarkupContainer createGuestNameContainer() {
     private TextField<String> createGuestNameField() {
         TextField<String> guestNameField = new RequiredTextField<String>("guestName");
         guestNameField.add(StringValidator.lengthBetween(3, 50));
+        guestNameField.add(new ValidationDisplayBehaviour());
         return guestNameField;
     }
 
@@ -134,12 +137,14 @@ private TextField<String> createGuestEmailField() {
         TextField<String> guestEmailField = new RequiredTextField<String>("guestEmail");
         guestEmailField.add(StringValidator.maximumLength(50));
         guestEmailField.add(EmailAddressValidator.getInstance());
+        guestEmailField.add(new ValidationDisplayBehaviour());
         return guestEmailField;
     }
 
     private TextArea<String> createCommentField() {
         TextArea<String> commentField = new TextArea<String>("comment");
         commentField.add(StringValidator.lengthBetween(10, 3000));
+        commentField.add(new ValidationDisplayBehaviour());
         commentField.setRequired(true);
         return commentField;
     }
@@ -236,7 +241,7 @@ public boolean isVisible() {
     }
 
     private FeedbackPanel createFeedbackPanel() {
-        FeedbackPanel feedbackPanel = new FeedbackPanel("feedback");
+        FeedbackPanel feedbackPanel = new PortalFeedbackPanel("feedback");
         feedbackPanel.setOutputMarkupId(true);
         return feedbackPanel;
     }
diff --git a/portal-core/src/main/java/org/devproof/portal/core/module/common/css/default.css b/portal-core/src/main/java/org/devproof/portal/core/module/common/css/default.css
index c73d6a73..3677a295 100644
--- a/portal-core/src/main/java/org/devproof/portal/core/module/common/css/default.css
+++ b/portal-core/src/main/java/org/devproof/portal/core/module/common/css/default.css
@@ -671,7 +671,6 @@ textarea.formtextarea {
 }
 
 input.error,
-select.error,
 textarea.error {
     background-color: #FFC700;
 }