Fix TextInputGroup

hpehl · hpehl · commit ba6cd71441e7 · 2024-08-15T12:58:06.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - Add `TreeViewItem.disabled(boolean)`
 - Add `Variable.utilVar(String, String...)`
 - Add `HasItems.items()`
+- Add `TextInputGroup.onChange(ChangeHandler<TextInputGroup, String> changeHandler)`
 
 ### Changed
 
diff --git a/components/src/main/java/org/patternfly/component/textinputgroup/TextInputGroup.java b/components/src/main/java/org/patternfly/component/textinputgroup/TextInputGroup.java
@@ -23,6 +23,7 @@
 import org.patternfly.component.chip.Chip;
 import org.patternfly.component.chip.ChipGroup;
 import org.patternfly.core.ObservableValue;
+import org.patternfly.handler.ChangeHandler;
 import org.patternfly.handler.CloseHandler;
 import org.patternfly.style.Modifiers.Disabled;
 
@@ -75,11 +76,7 @@ public static TextInputGroup searchInputGroup(String placeholder) {
                                     textEntered.change(false);
                                 })));
 
-        textEntered.subscribe((current, previous) -> {
-            if (current) {
-                setVisible(textInputGroup.utilities(), true);
-            }
-        });
+        textEntered.subscribe((current, previous) -> setVisible(textInputGroup.utilities(), current));
         textEntered.publish();
 
         return textInputGroup;
@@ -198,6 +195,14 @@ public TextInputGroup that() {
         return this;
     }
 
+    // ------------------------------------------------------ events
+
+    /** Delegates to {@link TextInputGroupMain#onChange(ChangeHandler)} */
+    public TextInputGroup onChange(ChangeHandler<TextInputGroup, String> changeHandler) {
+        main.onChange(changeHandler);
+        return this;
+    }
+
     // ------------------------------------------------------ api
 
     @SuppressWarnings("ConfusingMainMethod")
