Skip to content

Commit 893f082

Browse files
authored
Merge branch 'master' into mdragnev/add-row-transactions
2 parents 764034c + cabfdde commit 893f082

File tree

5 files changed

+8
-21
lines changed

5 files changed

+8
-21
lines changed

CHANGELOG.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ All notable changes for each version of this project will be documented in this
1313
- **Breaking Change** - `target` property in `PositionSettings` has been deprecated. You can set the attaching target for the component to show in `OverlaySettings` instead.
1414
- `IgxSelect`
1515
- Added `aria-labelledby` property for the items list container(marked as `role="listbox"`). This will ensure the users of assistive technologies will also know what the list items container is used for, upon opening.
16-
- `IgxDatePicker`
17-
- **Breaking Change** - Deprecated the `label` and `labelVisibility` properties.
16+
- `IgxDatePicker`
17+
- **Breaking Change** - Deprecated the `label` property.
1818

1919

2020
### New Features
@@ -40,7 +40,7 @@ All notable changes for each version of this project will be documented in this
4040
- ` IgxExcelExporterService`
4141
- Added `worksheetName` property to the `IgxExcelExporterOptions`, that allows setting the name of the worksheet.
4242
- `IgxDatePicker`
43-
- The `labelVisibility` and the `label` property have been deprecated and now a custom label is set by nesting a <label igxLabel></label> inside the <igx-date-picker><igx-date-picker> tags.
43+
- The the `label` property have been deprecated and a custom label can also be set by nesting a <label igxLabel></label> inside the <igx-date-picker><igx-date-picker> tags.
4444
- `IgxTimePicker`
4545
- Added a custom label functionality.
4646
- `IgxCalendar` and `IgxDatePicker` - new `showWeekNumbers` input, that allows showing of the week number at left side of content area.

projects/igniteui-angular/src/lib/date-picker/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ DatePicker with cancel and today buttons
4141
</igx-date-picker>
4242
```
4343

44-
The DatePicker's custom label can be set in the way shown below. If `labelVisibility` is set to `false` and a custom label is not used, a default one will be set.
44+
The DatePicker's custom label can be set in two ways, either by changing the `label` property or in the way shown below.
4545
````html
4646
<igx-date-picker>
4747
<label igxLabel>Custom label</label>

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
<ng-template #readOnlyDatePickerTemplate>
66
<igx-input-group (click)="openDialog()">
7-
<label igxLabel *ngIf="!_labelDirectiveUserTemplate&&!labelVisibility">Date</label>
87
<ng-container ngProjectAs="[igxLabel]" *ngTemplateOutlet="labelTemplate"></ng-container>
98
<igx-prefix>
109
<igx-icon>today</igx-icon>
@@ -24,7 +23,6 @@
2423

2524
<ng-template #editableDatePickerTemplate>
2625
<igx-input-group #editableInputGroup>
27-
<label igxLabel *ngIf="!_labelDirectiveUserTemplate&&!labelVisibility">Date</label>
2826
<ng-container ngProjectAs="[igxLabel]" *ngTemplateOutlet="labelTemplate"></ng-container>
2927
<igx-prefix (click)="onOpenClick($event)">
3028
<igx-icon>today</igx-icon>

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ describe('IgxDatePicker', () => {
159159
fixture.detectChanges();
160160

161161
label = fixture.debugElement.query(By.directive(IgxLabelDirective));
162-
expect(label.nativeElement.innerText).toBe('Date');
162+
expect(label).toBeNull();
163163
});
164164

165165
it('When update label property it should reflect on the label text of the datepicker', () => {
@@ -179,7 +179,7 @@ describe('IgxDatePicker', () => {
179179
fixture.detectChanges();
180180

181181
let label = fixture.debugElement.query(By.directive(IgxLabelDirective));
182-
expect(label.nativeElement.innerText).toBe('Date');
182+
expect(label).toBeNull();
183183

184184
fixture.componentInstance.labelVisibility = true;
185185
fixture.detectChanges();
@@ -207,9 +207,9 @@ describe('IgxDatePicker', () => {
207207
fixtureProjectedLabel.detectChanges();
208208
testComponent.customLabelVisibility = false;
209209
fixtureProjectedLabel.detectChanges();
210-
label = dom.query(By.directive(IgxLabelDirective)).nativeElement.innerText;
210+
label = dom.query(By.directive(IgxLabelDirective));
211211
fixtureProjectedLabel.detectChanges();
212-
expect(label).toEqual('Date');
212+
expect(label).toBeNull();
213213

214214
testComponent.customLabelVisibility = true;
215215
fixtureProjectedLabel.detectChanges();

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

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -190,18 +190,7 @@ export class IgxDatePickerComponent implements IDatePicker, ControlValueAccessor
190190
* By default the visibility is set to true.
191191
* @example
192192
* <igx-date-picker [labelVisibility]="false"></igx-date-picker>
193-
* @deprecated Use
194-
* ````html
195-
* <igx-date-picker>
196-
* <label igxLabel>Custom label</label>
197-
* </igx-date-picker>
198-
* ````
199-
* to set a custom label.
200193
*/
201-
@DeprecateProperty(`Deprecated. Use
202-
<igx-date-picker>
203-
<label igxLabel>Custom label</label>
204-
</igx-date-picker> to set a label.` )
205194
@Input()
206195
public labelVisibility = true;
207196

0 commit comments

Comments
 (0)