fixed a bug that clicking on the validation result icon will clear al…

…l other decorations on the same target node
jidesoft · Dec 5, 2013 · b69ea26 · b69ea26
1 parent 0ff7b65
commit b69ea26
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/module-validation/src/main/java/jidefx/scene/control/validation/ValidationUtils.java b/module-validation/src/main/java/jidefx/scene/control/validation/ValidationUtils.java
@@ -511,17 +511,17 @@ public void handle(ValidationEvent event) {
                     else {
                         label = new Label("", graphic);
                         TooltipEx.install(label, tooltip);
+                        resultDecorator = createDefaultDecorator(targetNode, label);
                         label.addEventHandler(MouseEvent.MOUSE_CLICKED, new EventHandler<MouseEvent>() {
                             @Override
                             public void handle(MouseEvent event) {
-                                DecorationUtils.uninstall(targetNode);
+                                DecorationUtils.uninstall(targetNode, resultDecorator);
                                 resultDecorator = null;
                                 targetNode.getProperties().remove(PROPERTY_VALIDATION_RESULT);
                                 targetNode.getParent().requestLayout();
                                 event.consume();
                             }
                         });
-                        resultDecorator = createDefaultDecorator(targetNode, label);
                         DecorationUtils.install(targetNode, resultDecorator);
                         targetNode.getParent().requestLayout();
                     }