aboutsummaryrefslogtreecommitdiffstats
path: root/src/com/itmill/toolkit/ui/AbstractField.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/com/itmill/toolkit/ui/AbstractField.java')
-rw-r--r--src/com/itmill/toolkit/ui/AbstractField.java8
1 files changed, 7 insertions, 1 deletions
diff --git a/src/com/itmill/toolkit/ui/AbstractField.java b/src/com/itmill/toolkit/ui/AbstractField.java
index 66eef1d6df..2fa622c7d8 100644
--- a/src/com/itmill/toolkit/ui/AbstractField.java
+++ b/src/com/itmill/toolkit/ui/AbstractField.java
@@ -143,6 +143,12 @@ public abstract class AbstractField extends AbstractComponent implements Field,
if (isRequired()) {
target.addAttribute("required", true);
}
+
+ // Hide the error indicator if needed
+ if (isRequired() && isEmpty() && getComponentError() == null
+ && getErrorMessage() != null) {
+ target.addAttribute("hideErrors", true);
+ }
}
/*
@@ -757,7 +763,7 @@ public abstract class AbstractField extends AbstractComponent implements Field,
// required fields, as in those cases user is aware of the problem.
// Furthermore, non-required empty fields are obviously correct.
ErrorMessage validationError = null;
- if (isValidationVisible() && !isEmpty()) {
+ if (isValidationVisible() && !(isEmpty() && !isRequired())) {
try {
validate();