@clr/angular 16.3.0 → 16.3.2

package/fesm2020/clr-angular.mjs

@@ -1955,12 +1955,12 @@ class ClrPopoverPositionService {
          *
          *   Note, more than 3 viewport violations and there isn't anything we can do to help. Also when there are two
          *   violations, we can't help if the violations are TOP+BOTTOM || LEFT+RIGHT => There is no transformation we
-         *   can make to the postion that will help.
+         *   can make to the position that will help.
          *
          *   Some examples:
          *   There is only one error and Primary axis is VERTICAL
          *   - this.handleVerticalAxisOneViolation has a switch that will use the error sum to apply the correct
-         *   transform to the postion based on the reduction of visibilityViolations.
+         *   transform to the position based on the reduction of visibilityViolations.
          *
          *   There are two errors and Primary axis is HORIZONTAL
          *   - handleHorizontalAxisTwoViolations has a switch that uses the error sum to apply both transforms needed to
@@ -1992,6 +1992,14 @@ class ClrPopoverPositionService {
         if (this.currentContentCoords.top < 0) {
             this.contentOffsets.yOffset += Math.abs(this.currentContentCoords.top);
         }
+        /**
+         * This detects the condition where the popover is flipped because it would violate the bottom of the viewport, but flipping it results in the
+         * popover rendering above the top of the body (y coordinate outside the body). In that event, it should be rendered within the body
+         * as much as possible, so this logic sets the top of popover to render touching the top of the body.
+         */
+        if (this.contentOffsets.yOffset + this.currentAnchorCoords.y < 0) {
+            this.contentOffsets.yOffset = 0 - this.currentContentCoords.top;
+        }
         return this.contentOffsets;
     }
     handleVerticalAxisOneViolation(errorSum) {
@@ -3817,16 +3825,14 @@ class ClrButtonGroup {
         this.menuButtons = tempInMenuButtons;
     }
     handleFocusOnMenuOpen() {
-        if (this.menuButtons.length) {
-            this.toggleService.popoverVisible.pipe(takeUntil(this.destroy$)).subscribe(visible => {
-                if (visible) {
-                    this.focusHandler.initialize({
-                        menu: this.menu.nativeElement,
-                        menuToggle: this.menuToggle.nativeElement,
-                    });
-                }
-            });
-        }
+        this.toggleService.popoverVisible.pipe(takeUntil(this.destroy$)).subscribe(visible => {
+            if (visible) {
+                this.focusHandler.initialize({
+                    menu: this.menu.nativeElement,
+                    menuToggle: this.menuToggle.nativeElement,
+                });
+            }
+        });
     }
 }
 ClrButtonGroup.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "15.2.2", ngImport: i0, type: ClrButtonGroup, deps: [{ token: ButtonInGroupService }, { token: ClrPopoverToggleService }, { token: ClrCommonStringsService }, { token: ClrDestroyService }, { token: ButtonGroupFocusHandler }], target: i0.ɵɵFactoryTarget.Component });
@@ -3925,7 +3931,7 @@ class ClrLoadingButton {
 }
 ClrLoadingButton.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "15.2.2", ngImport: i0, type: ClrLoadingButton, deps: [{ token: i0.ElementRef }, { token: i0.Renderer2 }], target: i0.ɵɵFactoryTarget.Component });
 ClrLoadingButton.ɵcmp = i0.ɵɵngDeclareComponent({ minVersion: "14.0.0", version: "15.2.2", type: ClrLoadingButton, selector: "button[clrLoading]", inputs: { disabled: "disabled" }, outputs: { clrLoadingChange: "clrLoadingChange" }, host: { properties: { "attr.disabled": "disabled? '' : null" } }, providers: [{ provide: LoadingListener, useExisting: ClrLoadingButton }], ngImport: i0, template: `
-    <ng-container [ngSwitch]="state">
+    <span @parent [ngSwitch]="state">
       <span *ngSwitchCase="buttonState.LOADING">
         <span @spinner class="spinner spinner-inline"></span>
       </span>
@@ -3939,8 +3945,13 @@ ClrLoadingButton.ɵcmp = i0.ɵɵngDeclareComponent({ minVersion: "14.0.0", versi
       <span *ngSwitchCase="buttonState.DEFAULT" @defaultButton>
         <ng-content></ng-content>
       </span>
-    </ng-container>
+    </span>
   `, isInline: true, dependencies: [{ kind: "directive", type: i5.NgSwitch, selector: "[ngSwitch]", inputs: ["ngSwitch"] }, { kind: "directive", type: i5.NgSwitchCase, selector: "[ngSwitchCase]", inputs: ["ngSwitchCase"] }], animations: [
+        trigger('parent', [
+            // Skip :enter animation on first render.
+            // The button text/content should only be faded when transitioning to or from a non-default state.
+            transition(':enter', []),
+        ]),
         trigger('defaultButton', [
             transition(':enter', [style({ opacity: 0 }), animate('200ms 100ms ease-in', style({ opacity: 1 }))]),
             // TODO: see if we can get leave animation to work before spinner's enter animation
@@ -3968,7 +3979,7 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "15.2.2", ngImpor
             args: [{
                     selector: 'button[clrLoading]',
                     template: `
-    <ng-container [ngSwitch]="state">
+    <span @parent [ngSwitch]="state">
       <span *ngSwitchCase="buttonState.LOADING">
         <span @spinner class="spinner spinner-inline"></span>
       </span>
@@ -3982,10 +3993,15 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "15.2.2", ngImpor
       <span *ngSwitchCase="buttonState.DEFAULT" @defaultButton>
         <ng-content></ng-content>
       </span>
-    </ng-container>
+    </span>
   `,
                     providers: [{ provide: LoadingListener, useExisting: ClrLoadingButton }],
                     animations: [
+                        trigger('parent', [
+                            // Skip :enter animation on first render.
+                            // The button text/content should only be faded when transitioning to or from a non-default state.
+                            transition(':enter', []),
+                        ]),
                         trigger('defaultButton', [
                             transition(':enter', [style({ opacity: 0 }), animate('200ms 100ms ease-in', style({ opacity: 1 }))]),
                             // TODO: see if we can get leave animation to work before spinner's enter animation
@@ -8402,7 +8418,7 @@ class DateIOService {
         return '';
     }
     getDateValueFromDateString(date) {
-        if (!date) {
+        if (!date || typeof date !== 'string') {
             return null;
         }
         const dateParts = date.match(USER_INPUT_REGEX);
@@ -9608,9 +9624,11 @@ class ClrDateInput extends WrappedFormControl {
     }
     set min(dateString) {
         this.dateIOService.setMinDate(dateString);
+        this.triggerControlValidation();
     }
     set max(dateString) {
         this.dateIOService.setMaxDate(dateString);
+        this.triggerControlValidation();
     }
     get disabled() {
         if (this.dateFormControlService) {
@@ -9675,6 +9693,13 @@ class ClrDateInput extends WrappedFormControl {
             this.focusService.focused = focus;
         }
     }
+    triggerControlValidation() {
+        if (this.datepickerHasFormControl()) {
+            // Set `emitEvent` to false to prevent unnecessary value change event. Status change event will be emitted by `setErrors` below.
+            this.control.control?.updateValueAndValidity({ emitEvent: false });
+            this.control.control?.setErrors(this.control.control.errors);
+        }
+    }
     populateServicesFromContainerComponent() {
         if (!this.container) {
             this.dateIOService = this.getProviderFromContainer(DateIOService);
@@ -19954,7 +19979,7 @@ class ClrAlert {
         }
         const isCurrentAlert = this.multiAlertService?.currentAlert === this;
         this._closed = true;
-        if (this.multiAlertService) {
+        if (this.multiAlertService?.activeAlerts) {
             this.multiAlertService.close(isCurrentAlert);
         }
         this._closedChanged.emit(true);