From 14d59b3fccf8f768777ce89078fee0601bbbdcf7 Mon Sep 17 00:00:00 2001 From: Max Okorokov Date: Thu, 9 Jan 2020 12:15:33 +0100 Subject: [PATCH] chore: cleanup datepicker `(select)` deprecation --- .../datepicker/overview/datepicker-overview.component.html | 2 +- src/datepicker/datepicker-service.ts | 3 --- src/datepicker/datepicker.ts | 3 ++- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/demo/src/app/components/datepicker/overview/datepicker-overview.component.html b/demo/src/app/components/datepicker/overview/datepicker-overview.component.html index 7bfc396432..087d829391 100644 --- a/demo/src/app/components/datepicker/overview/datepicker-overview.component.html +++ b/demo/src/app/components/datepicker/overview/datepicker-overview.component.html @@ -101,7 +101,7 @@

Handling the popup

- Alternatively you could use the (dateSelect) outputs. + Alternatively you could use the (dateSelect) output. The difference from ngModel is that outputs will continue emitting the same value, if user clicks on the same date. NgModel will do it only once.

diff --git a/src/datepicker/datepicker-service.ts b/src/datepicker/datepicker-service.ts index 7769f10e04..8262d82776 100644 --- a/src/datepicker/datepicker-service.ts +++ b/src/datepicker/datepicker-service.ts @@ -45,9 +45,6 @@ export class NgbDatepickerService { get dateSelect$(): Observable { return this._dateSelect$.pipe(filter(date => date !== null)); } - /** @deprecated please use `dateSelect$` */ - get select$(): Observable { return this.dateSelect$; } - set dayTemplateData(dayTemplateData: NgbDayTemplateData) { if (this._state.dayTemplateData !== dayTemplateData) { this._nextState({dayTemplateData}); diff --git a/src/datepicker/datepicker.ts b/src/datepicker/datepicker.ts index 8480822773..96b7c7d556 100644 --- a/src/datepicker/datepicker.ts +++ b/src/datepicker/datepicker.ts @@ -280,7 +280,8 @@ export class NgbDatepicker implements OnDestroy, * * The payload of the event is currently selected `NgbDate`. * - * @deprecated please use 'dateSelect' event + * Please use 'dateSelect' output instead, this will be deprecated in version 6.0 due to collision with native + * 'select' event. */ @Output() select = this.dateSelect;