Skip to content

Commit e4794ce

Browse files
didimmovasimeonoff
andauthored
fix(date-range): align 'to' label for daterange picker (#16492)
Co-authored-by: Simeon Simeonoff <[email protected]>
1 parent df52d17 commit e4794ce

File tree

4 files changed

+48
-6
lines changed

4 files changed

+48
-6
lines changed

projects/igniteui-angular/core/src/core/styles/components/date-range-picker/_date-range-picker-theme.scss

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,12 @@
66
/// @param {Map} $theme - The theme used to style the component.
77
@mixin date-range-picker($theme) {
88
@include css-vars($theme);
9+
$variant: map.get($theme, '_meta', 'theme');
910

1011
%igx-date-range-picker {
12+
@include sizable();
13+
--input-group-size: #{map.get($theme, 'size')};
14+
--component-size: var(--ig-size, #{var-get($theme, 'default-size')});
1115
display: flex;
1216

1317
> igx-icon {
@@ -32,6 +36,8 @@
3236
igx-date-range-end,
3337
%igx-date-range-picker__start,
3438
%igx-date-range-picker__end {
39+
--size: var(--input-group-size) !important;
40+
3541
flex: 1 0 0%;
3642
}
3743

@@ -40,6 +46,17 @@
4046
align-items: center;
4147
color: var-get($theme, 'label-color');
4248
margin: 0 rem(8px);
49+
height: var(--input-group-size);
50+
51+
@if $variant != 'material' {
52+
align-self: flex-end;
53+
54+
&.input-has-hint {
55+
align-self: center;
56+
}
57+
} @else {
58+
align-self: flex-start;
59+
}
4360
}
4461

4562
%igx-date-range-picker-buttons {

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

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,24 @@ import { IgxCalendarContainerComponent } from '../date-picker/calendar-container
4343
import { PickerBaseDirective } from '../date-picker/picker-base.directive';
4444
import { IgxOverlayOutletDirective } from 'igniteui-angular/directives';
4545
import {
46-
IgxInputDirective, IgxInputGroupComponent, IgxInputGroupType, IgxInputState,
47-
IgxLabelDirective, IGX_INPUT_GROUP_TYPE, IgxSuffixDirective,
46+
IgxInputDirective,
47+
IgxInputGroupComponent,
48+
IgxInputGroupType,
49+
IgxInputState,
50+
IgxLabelDirective,
51+
IGX_INPUT_GROUP_TYPE,
52+
IgxSuffixDirective,
4853
IgxPrefixDirective,
49-
IgxReadOnlyInputDirective
54+
IgxReadOnlyInputDirective,
55+
IgxHintDirective
5056
} from 'igniteui-angular/input-group';
51-
import { IgxDateRangeEndComponent, IgxDateRangeInputsBaseComponent, IgxDateRangeSeparatorDirective, IgxDateRangeStartComponent, DateRangePickerFormatPipe } from './date-range-picker-inputs.common';
57+
import {
58+
IgxDateRangeEndComponent,
59+
IgxDateRangeInputsBaseComponent,
60+
IgxDateRangeSeparatorDirective,
61+
IgxDateRangeStartComponent,
62+
DateRangePickerFormatPipe,
63+
} from './date-range-picker-inputs.common';
5264
import { IgxIconComponent } from 'igniteui-angular/icon';
5365
import { fadeIn, fadeOut } from 'igniteui-angular/animations';
5466

@@ -420,6 +432,9 @@ export class IgxDateRangePickerComponent extends PickerBaseDirective
420432
@ContentChild(IgxLabelDirective)
421433
public label: IgxLabelDirective;
422434

435+
@ContentChild(IgxHintDirective)
436+
public hint: IgxHintDirective;
437+
423438
@ContentChild(IgxPickerActionsDirective)
424439
public pickerActions: IgxPickerActionsDirective;
425440

@@ -550,7 +565,9 @@ export class IgxDateRangePickerComponent extends PickerBaseDirective
550565

551566
/** @hidden @internal */
552567
public get separatorClass(): string {
553-
return 'igx-date-range-picker__label';
568+
const classes = ['igx-date-range-picker__label'];
569+
if (this.hint) classes.push('input-has-hint');
570+
return classes.join(' ');
554571
}
555572

556573
protected override get toggleContainer(): HTMLElement | undefined {

src/app/date-range/date-range.sample.html

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@ <h6>Angular Date Range Picker, two inputs, template-driven form</h6>
4646
type="text"
4747
[placeholder]="properties.placeholder"
4848
/>
49-
<igx-hint>Helper text</igx-hint>
49+
@if (properties.hint) {
50+
<span igxHint>{{ properties.hint }}</span>
51+
}
5052
</igx-date-range-start>
5153
<igx-date-range-end>
5254
<igx-picker-toggle igxPrefix>

src/app/date-range/date-range.sample.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,12 @@ export class DateRangeSampleComponent {
148148
defaultValue: 'MM/dd/yyyy'
149149
}
150150
},
151+
hint: {
152+
label: 'Hint Text',
153+
control: {
154+
type: 'text'
155+
}
156+
},
151157
displayFormat: {
152158
label: 'Display Format',
153159
control: {

0 commit comments

Comments
 (0)