Fixed #14036 - ListBox | Refactor onClick emitter

primefaces · Nov 6, 2023 · fc68381 · fc68381 · vercel · Nov 6, 2023
1 parent 26443f2
commit fc68381
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/app/components/listbox/listbox.ts b/src/app/components/listbox/listbox.ts
@@ -750,12 +750,6 @@ export class Listbox implements AfterContentInit, OnInit, ControlValueAccessor,
         this.modelValue.set(value);
         this.onModelChange(value);
 
-        if(event && event.type == 'click') {
-            this.onClick.emit({
-                originalEvent: event,
-                value: value
-            });
-        }
         this.onChange.emit({ originalEvent: event, value: this.value });
     }
 
@@ -768,6 +762,8 @@ export class Listbox implements AfterContentInit, OnInit, ControlValueAccessor,
             return;
         }
 
+        event && this.onClick.emit({originalEvent: event, value: option});
+
         this.multiple ? this.onOptionSelectMultiple(event, option) : this.onOptionSelectSingle(event, option);
         this.optionTouched = false;
         index !== -1 && this.focusedOptionIndex.set(index);