Skip to content

Commit ab064a4

Browse files
committedJun 7, 2019
chore(*): more code improvements #4647
1 parent 4bb4d8e commit ab064a4

File tree

6 files changed

+13
-10
lines changed

6 files changed

+13
-10
lines changed
 

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

+2-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ import { KEYS, CancelableBrowserEventArgs } from '../core/utils';
5454
import { IgxDatePickerTemplateDirective, IgxDatePickerActionsDirective } from './date-picker.directives';
5555
import { IgxCalendarContainerComponent } from './calendar-container.component';
5656
import { InteractionMode } from '../core/enums';
57-
import { getViewportRect } from '../services/overlay/utilities';
5857
import { fadeIn, fadeOut } from '../animations/fade';
5958
import { DeprecateProperty } from '../core/deprecateDecorators';
6059

@@ -540,6 +539,7 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
540539
public outlet: IgxOverlayOutletDirective | ElementRef;
541540

542541
/**
542+
* @deprecated Use 'onOpened' instead.
543543
*An event that is emitted when the `IgxDatePickerComponent` calendar is opened.
544544
*```typescript
545545
*public open(event){
@@ -561,6 +561,7 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
561561
public onOpened = new EventEmitter<IgxDatePickerComponent>();
562562

563563
/**
564+
* @deprecated Use 'onClosed' instead.
564565
*"An event that is emitted when the `IgxDatePickerComponent` is closed.
565566
*```typescript
566567
*public close(event){

‎projects/igniteui-angular/src/lib/date-picker/date-picker.directives.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Directive, TemplateRef, Input } from '@angular/core';
1+
import { Directive, TemplateRef } from '@angular/core';
22

33
@Directive({
44
selector: '[igxDatePickerTemplate]'

‎projects/igniteui-angular/src/lib/time-picker/time-picker.common.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export interface IgxTimePickerBase {
2828
scrollHourIntoView(item: string): void;
2929
scrollMinuteIntoView(item: string): void;
3030
scrollAmPmIntoView(item: string): void;
31-
hideOverlay(): void;
31+
close(): void;
3232
parseMask(preserveAmPm?: boolean): string;
3333
}
3434

‎projects/igniteui-angular/src/lib/time-picker/time-picker.component.ts

+7-5
Original file line numberDiff line numberDiff line change
@@ -402,6 +402,7 @@ export class IgxTimePickerComponent implements
402402
public onValidationFailed = new EventEmitter<IgxTimePickerValidationFailedEventArgs>();
403403

404404
/**
405+
* @deprecated Use 'onOpened' instead.
405406
* Emitted when a timePicker is being opened.
406407
* ```html
407408
*@ViewChild("toast")
@@ -427,6 +428,7 @@ export class IgxTimePickerComponent implements
427428
public onOpened = new EventEmitter<IgxTimePickerComponent>();
428429

429430
/**
431+
* @deprecated Use 'onClosed' instead.
430432
* Emitted when a timePicker is being closed.
431433
*/
432434
@DeprecateProperty(`'onClose' @Output property is deprecated. Use 'onClosed' instead.`)
@@ -1428,7 +1430,7 @@ export class IgxTimePickerComponent implements
14281430
public okButtonClick(): boolean {
14291431
const time = this._getSelectedTime();
14301432
if (this._isValueValid(time)) {
1431-
this.hideOverlay();
1433+
this.close();
14321434
this.value = time;
14331435
return true;
14341436
} else {
@@ -1455,7 +1457,7 @@ export class IgxTimePickerComponent implements
14551457
this.displayValue = this._formatTime(this.value, this.format);
14561458
}
14571459

1458-
this.hideOverlay();
1460+
this.close();
14591461

14601462
this.selectedHour = this._prevSelectedHour;
14611463
this.selectedMinute = this._prevSelectedMinute;
@@ -1511,10 +1513,10 @@ export class IgxTimePickerComponent implements
15111513
* ```
15121514
* ```typescript
15131515
* @ViewChild('timePicker', { read: IgxTimePickerComponent }) picker: IgxTimePickerComponent;
1514-
* picker.hideOverlay();
1516+
* picker.close();
15151517
* ```
15161518
*/
1517-
public hideOverlay(): void {
1519+
public close(): void {
15181520
this.toggleRef.close();
15191521
}
15201522

@@ -1549,7 +1551,7 @@ export class IgxTimePickerComponent implements
15491551
this.onValueChanged.emit(args);
15501552
}
15511553
} else {
1552-
this.hideOverlay();
1554+
this.close();
15531555
}
15541556
}
15551557

‎projects/igniteui-angular/src/lib/time-picker/time-picker.directives.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ export class IgxItemListDirective {
157157
event.preventDefault();
158158

159159
if (this.timePicker.mode === InteractionMode.DropDown) {
160-
this.timePicker.hideOverlay();
160+
this.timePicker.close();
161161
return;
162162
}
163163
this.timePicker.okButtonClick();

‎src/app/time-picker/time-picker.sample.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,6 @@ export class TimePickerSampleComponent implements AfterViewInit {
6464

6565
public selectToday(picker: IgxTimePickerComponent) {
6666
picker.value = new Date(Date.now());
67-
picker.hideOverlay();
67+
picker.close();
6868
}
6969
}

0 commit comments

Comments
 (0)