Skip to content

Commit 459b845

Browse files
fix(IgxDatePicker): return when IE triggers input before focus
1 parent b70a776 commit 459b845

File tree

1 file changed

+13
-5
lines changed

1 file changed

+13
-5
lines changed

projects/igniteui-angular/src/lib/date-picker/date-picker.component.ts

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,9 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
392392
private _cdr: ChangeDetectorRef,
393393
private _moduleRef: NgModuleRef<any>,
394394
private _injector: Injector,
395-
private _renderer: Renderer2) { }
395+
private _renderer: Renderer2) {
396+
}
397+
396398

397399
/**
398400
* Gets the input group template.
@@ -792,7 +794,7 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
792794
return this._inputGroup || this._inputGroupUserTemplate || null;
793795
}
794796

795-
/** @hidden @internal */
797+
/** @hidden @internal */
796798
public get inputDirective(): IgxInputDirective {
797799
return this._inputDirective || this._inputDirectiveUserTemplates.first || null;
798800
}
@@ -885,7 +887,7 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
885887

886888
this._inputDirectiveUserTemplates.changes.subscribe(() => {
887889
this.attachTemplateBlur();
888-
});
890+
});
889891
this.attachTemplateBlur();
890892
}
891893

@@ -901,8 +903,8 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
901903
this.rawDateString = (res.target as HTMLInputElement).value;
902904
this.onBlur(res, false);
903905
});
904-
// TODO: Refactor custom template handling.
905-
// Revise blur handling when custom template is passed
906+
// TODO: Refactor custom template handling.
907+
// Revise blur handling when custom template is passed
906908
}
907909
}
908910

@@ -1141,6 +1143,12 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
11411143

11421144
/** @hidden @internal */
11431145
public onInput(event) {
1146+
/**
1147+
* Fix for #8165 until refactoring (#6483).
1148+
* The IgxDateTimeEditor will be used to handle all inputs, i.e. this handler will be removed.
1149+
* It extends the IgxMaskDirective which contains logic that handles this issue.
1150+
*/
1151+
if (isIE() && !this._isInEditMode && !this.inputGroup.isFocused) { return; }
11441152
const targetValue = event.target.value;
11451153
const cursorPosition = this._getCursorPosition();
11461154
const checkInput = DatePickerUtil.checkForCompleteDateInput(this.dateFormatParts, targetValue);

0 commit comments

Comments
 (0)