Merge pull request #308 from bohdan-harniuk/306-add-highlighting-to-inputs-on-error

306: Added highlighting in error style to JComponent elements

Author: Vitaliy

src/com/magento/idea/magento2plugin/actions/generation/dialog/AbstractDialog.java

@@ -5,6 +5,7 @@
 
 package com.magento.idea.magento2plugin.actions.generation.dialog;
 
+import com.magento.idea.magento2plugin.actions.generation.dialog.util.HighlightDialogFieldOnErrorUtil;
 import com.magento.idea.magento2plugin.actions.generation.dialog.validator.annotation.FieldValidation;
 import com.magento.idea.magento2plugin.actions.generation.dialog.validator.annotation.FieldValidations;
 import com.magento.idea.magento2plugin.actions.generation.dialog.validator.rule.ValidationRule;
@@ -74,6 +75,7 @@ protected boolean validateFormFields() {
                     if (errorMessageFieldValidationRuleMap.containsKey(field)
                             && errorMessageFieldValidationRuleMap.get(field).containsKey(rule)) {
                         showErrorMessage(errorMessageFieldValidationRuleMap.get(field).get(rule));
+                        highlightFieldWithErrorStyle(field);
                     }
                     return false;
                 }
@@ -186,4 +188,17 @@ private String resolveFieldValueByComponentType(final Object field) {
         }
         return null;
     }
+
+    /**
+     * Highlight field with error style.
+     *
+     * @param field Object
+     */
+    private void highlightFieldWithErrorStyle(final Object field) {
+        if (field instanceof JTextField) {
+            HighlightDialogFieldOnErrorUtil.execute((JTextField) field);
+        } else if (field instanceof JComboBox) {
+            HighlightDialogFieldOnErrorUtil.execute((JComboBox) field);
+        }
+    }
 }

src/com/magento/idea/magento2plugin/actions/generation/dialog/util/HighlightDialogFieldOnErrorUtil.java

@@ -0,0 +1,37 @@
+package com.magento.idea.magento2plugin.actions.generation.dialog.util;
+
+import java.awt.Color;
+import java.awt.event.FocusEvent;
+import java.awt.event.FocusListener;
+import javax.swing.BorderFactory;
+import javax.swing.JComponent;
+
+public final class HighlightDialogFieldOnErrorUtil {
+    private static final Color ERROR_BORDER_COLOR = new Color(252, 119, 83);
+    private static final Color ERROR_BACKGROUND_COLOR = new Color(252, 119, 83, 15);
+
+    private HighlightDialogFieldOnErrorUtil() {}
+
+    /**
+     * Add error highlighting for JComponent.
+     *
+     * @param field JComponent
+     */
+    public static void execute(final JComponent field) {
+        final Color defaultBackgroundColor = field.getBackground();
+
+        field.setBorder(BorderFactory.createLineBorder(ERROR_BORDER_COLOR));
+        field.setBackground(ERROR_BACKGROUND_COLOR);
+
+        field.addFocusListener(new FocusListener() {
+            @Override
+            public void focusGained(final FocusEvent event) {
+                field.setBorder(null);
+                field.setBackground(defaultBackgroundColor);
+            }
+
+            @Override
+            public void focusLost(FocusEvent e) {}//NOPMD
+        });
+    }
+}