@dereekb/dbx-form 13.10.8 → 13.10.9

package/fesm2022/dereekb-dbx-form.mjs

@@ -6051,7 +6051,12 @@ class DbxForgeDateTimeFieldComponent {
                 return;
             const currentDateCtrl = this.dateCtrl.value;
             if (!currentDateCtrl || !isSameDateDay(currentDateCtrl, date)) {
-                this.dateCtrl.setValue(date, { emitEvent: false });
+                // Emit valueChanges (default) so currentDate$ — and therefore date$/dateValue$/_rawDateTimeDate$
+                // — see the inbound date. Without this emission, rawDateTime$.combineLatest is starved on the
+                // date side and a user-typed time edit can never propagate back to the form value.
+                // The outbound pipeline's `filter(!dbxDateTimeIsSameDateTimeFieldValue(x, currentValue))`
+                // prevents the resulting _updateTime tick from causing a feedback write.
+                this.dateCtrl.setValue(date);
             }
             // Do not overwrite time control while user is actively editing it
             if (!isTimeFocused) {
@@ -6061,10 +6066,14 @@ class DbxForgeDateTimeFieldComponent {
                 }
             }
         });
-        // Main output subscription: timeOutput$ → field value
-        this._sub.subscription = this.valueInSystemTimezone$
-            .pipe(combineLatestWith$1(this.timezoneInstance$.pipe(map$1((tz) => dbxDateTimeOutputValueFactory(this.valueMode(), tz)))), throttleTime$1(TIME_OUTPUT_THROTTLE_TIME$2, undefined, { leading: false, trailing: true }), switchMap$1(([currentValue, valueFactory]) => {
-            return this.timeOutput$.pipe(throttleTime$1(TIME_OUTPUT_THROTTLE_TIME$2 * 2, undefined, { leading: false, trailing: true }), skipAllInitialMaybe(), distinctUntilChanged$1(isSameDateHoursAndMinutes), map$1((x) => valueFactory(x)), filter$1((x) => !dbxDateTimeIsSameDateTimeFieldValue(x, currentValue)));
+        // Main output subscription: timeOutput$ → field value.
+        // The filter must compare in storage form against the raw fieldValue. Comparing against
+        // valueInSystemTimezone$ (display-shifted) is wrong: with a non-zero system↔target tz shift,
+        // a freshly picked time can have the same timestamp as the previous valueInSystemTimezone$
+        // and be incorrectly dropped (e.g. CDT system + UTC field tz: pick 12pm → pick 5pm collides).
+        this._sub.subscription = this.fieldValue$
+            .pipe(combineLatestWith$1(this.timezoneInstance$.pipe(map$1((tz) => dbxDateTimeOutputValueFactory(this.valueMode(), tz)))), throttleTime$1(TIME_OUTPUT_THROTTLE_TIME$2, undefined, { leading: false, trailing: true }), switchMap$1(([currentRawValue, valueFactory]) => {
+            return this.timeOutput$.pipe(throttleTime$1(TIME_OUTPUT_THROTTLE_TIME$2 * 2, undefined, { leading: false, trailing: true }), skipAllInitialMaybe(), distinctUntilChanged$1(isSameDateHoursAndMinutes), map$1((x) => valueFactory(x)), filter$1((x) => !dbxDateTimeIsSameDateTimeFieldValue(x, currentRawValue)));
         }))
             .subscribe((value) => {
             this._setFieldValue(value);