chrv-components 1.12.222 → 1.12.224

package/fesm2022/chrv-components.mjs

@@ -4133,6 +4133,12 @@ class ChrPopoverDirective {
                 return mayBeTemplate;
         }, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "targetElementId" }] : /* istanbul ignore next */ []));
+        this.isNativePopoverTrigger = computed(() => {
+            const host = this.hostElement();
+            return (host instanceof HTMLButtonElement ||
+                (host instanceof HTMLInputElement && host.type === 'button'));
+        }, /* @ts-ignore */
+        ...(ngDevMode ? [{ debugName: "isNativePopoverTrigger" }] : /* istanbul ignore next */ []));
         this.updateLastKnownMousePosition = (event) => {
             this.lastKnownMousePosition$.next({
                 x: event.clientX,
@@ -4198,6 +4204,8 @@ class ChrPopoverDirective {
             return this.isInsideRect(x, y, rect);
         };
         effect(() => {
+            if (!this.isNativePopoverTrigger())
+                return;
             const open = this.isOpen();
             if (open)
                 this.showPopover();
@@ -4235,9 +4243,7 @@ class ChrPopoverDirective {
         this.renderer.addClass(this.targetElement(), 'card');
         this.renderer.addClass(this.targetElement(), 'glass');
         this.renderer.setStyle(this.targetElement(), 'border', 'none');
-        const isNativePopoverTrigger = host instanceof HTMLButtonElement ||
-            (host instanceof HTMLInputElement && host.type === 'button');
-        if (!this.chrPopoverOnHover() && !isNativePopoverTrigger) {
+        if (!this.chrPopoverOnHover() && !this.isNativePopoverTrigger()) {
             this.renderer.listen(this.hostElement(), 'click', () => {
                 if (this.isOpen()) {
                     this.isOpen.set(false);
@@ -4311,6 +4317,7 @@ class ChrInputComponent {
     // public valueChanges = output<any>();
     /* CODE */
     constructor() {
+        this.destroyRef = inject(DestroyRef);
         /**
          * Label of the input
          */
@@ -4483,7 +4490,7 @@ class ChrInputComponent {
         // The Behavior Subject always has an initial value. Which means that subscribing to it will make it look like the input has a new value (initially null).
         // Which is false, because we're only instanciating the object. This is why we need to skip the first value. Otherwise the form will receive a random null when this
         // observable is subscribed to and set the value to said null.
-        skip(1), debounceTime$1(this.debounceTime()))
+        takeUntilDestroyed(this.destroyRef), skip(1), debounceTime$1(this.debounceTime()))
             .subscribe((res) => {
             this.debouncedInput.emit(res);
             // If the value is an empty string, set the value to null to trigger validation and prevent an inconsitent value to be set (ex: for number type input)
@@ -4504,15 +4511,17 @@ class ChrInputComponent {
             this.isFocused.set(true);
         });
         this.ngControl()
-            ?.control?.events.pipe(filter((e) => e instanceof TouchedChangeEvent), map((e) => e.touched), 
+            ?.control?.events.pipe(takeUntilDestroyed(this.destroyRef), filter((e) => e instanceof TouchedChangeEvent), map((e) => e.touched), 
         // Start with will inject a starting value, it MUST be used after eventual transformation (here Event -> boolean)
         // and contain the type of the final data
         startWith(this.ngControl()?.control?.touched ?? false))
             .subscribe((e) => this.isTouched.set(e));
         this.ngControl()
-            ?.control?.events.pipe(filter((e) => e instanceof StatusChangeEvent), map((e) => e.status === 'DISABLED'), startWith(this.ngControl()?.control?.disabled ?? false))
+            ?.control?.events.pipe(takeUntilDestroyed(this.destroyRef), filter((e) => e instanceof StatusChangeEvent), map((e) => e.status === 'DISABLED'), startWith(this.ngControl()?.control?.disabled ?? false))
             .subscribe((e) => this.isDisabled.set(e));
-        this.ngControl()?.statusChanges?.subscribe((data) => {
+        this.ngControl()
+            ?.statusChanges?.pipe(takeUntilDestroyed(this.destroyRef))
+            .subscribe((data) => {
             this.ensureState(data);
         });
         this.displayValue.subscribe((v) => {