ngx-datex 1.0.2 → 1.0.3

package/fesm2022/ngx-datex.mjs

@@ -2808,6 +2808,7 @@ class NgxDatex {
     oldStartDate = null;
     oldEndDate = null;
     previousRightTime = null;
+    _isApplying = false; // Flag to prevent double emission during apply
     // Public getters for current dates
     get currentStartDate() {
         return this._internalStartDate;
@@ -3169,6 +3170,11 @@ class NgxDatex {
      * Used when calendar is closed without explicit apply (ESC, click outside, etc.)
      */
     closeCalendarWithChangeCheck() {
+        // Don't emit if we're in the middle of an apply operation
+        if (this._isApplying) {
+            this.closeCalendar();
+            return;
+        }
         // Check for changes using appropriate precision based on timePicker
         if (this.oldStartDate && this.oldEndDate) {
             const precision = this.timePicker() ? 'minute' : 'day';
@@ -3195,6 +3201,8 @@ class NgxDatex {
     apply() {
         if (!this.canApplyValue())
             return;
+        // Set flag to prevent double emission
+        this._isApplying = true;
         // Ensure time values are applied to the dates before closing
         if (this.timePicker()) {
             const startTime = this._startTime();
@@ -3224,7 +3232,10 @@ class NgxDatex {
         // Emit value change when applying (following vanilla daterangepicker behavior)
         this.emitValueChange();
         this.updateElement();
+        // Close calendar without checking for changes since we already applied them
         this.closeCalendar();
+        // Reset flag after closing
+        this._isApplying = false;
     }
     /**
      * Cancels the current selection and reverts to the previous values.
@@ -3367,7 +3378,12 @@ class NgxDatex {
         this.apply();
     }
     selectCustomRange() {
-        // Custom range selection logic - mainly visual
+        // Following vanilla daterangepicker: custom range selection just shows calendars
+        // It doesn't auto-apply like predefined ranges
+        // The user needs to manually select dates and click Apply
+        // Just ensure calendars are visible - no auto-apply for custom range
+        // This matches vanilla daterangepicker behavior where custom range
+        // requires manual Apply button click
     }
     /**
      * Gets the current checkbox state.