Skip to content

Commit fb9fc00

Browse files
committed
refactor(radio-group): styling improvements
1 parent cc99ed4 commit fb9fc00

File tree

2 files changed

+16
-14
lines changed

2 files changed

+16
-14
lines changed

projects/igniteui-angular/src/lib/core/styles/components/radio/_radio-theme.scss

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -597,31 +597,33 @@
597597
}
598598

599599
%radio-group-display {
600-
display: block;
600+
display: grid;
601+
column-gap: $horizontal-group-margin;
602+
align-items: start;
601603
width: max-content;
604+
}
602605

603-
%radio-display:not(:last-of-type) {
604-
margin-inline-end: $horizontal-group-margin;
605-
}
606-
606+
%radio-group-display:not(%radio-group-display--vertical) {
607607
[igxLabel] {
608-
display: block;
609608
margin-block-end: rem(16px);
610609
}
610+
611+
igx-radio {
612+
grid-row: 2;
613+
}
611614
}
612615

613616
%radio-group-display--vertical {
614-
display: flex;
615-
flex-flow: column;
617+
row-gap: $vertical-group-margin;
616618

617-
%radio-display:not(:last-of-type) {
618-
margin-inline-end: 0;
619-
margin-block-end: $vertical-group-margin;
619+
[igxLabel] {
620+
margin-block-end: if($variant == 'material', rem(16px), rem(8px));
620621
}
621622

622623
&%radio-group-display--before {
624+
igx-radio,
623625
[igxLabel] {
624-
text-align: end;
626+
justify-self: flex-end;
625627
}
626628
}
627629
}

src/app/input-controls/input-controls.sample.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,13 +62,13 @@ <h4 class="sample-title">Angular Inputs</h4>
6262
[checked]="properties.checked"
6363
[disabled]="properties.disabled"
6464
[invalid]="properties.invalid"
65-
>Radio Button 1</igx-radio>
65+
>Radio Button</igx-radio>
6666

6767
<igx-radio
6868
[disabled]="properties.disabled"
6969
[invalid]="properties.invalid"
7070
[labelPosition]="properties.labelPosition"
71-
>Radio Button 2</igx-radio>
71+
>Second Radio Button</igx-radio>
7272
</igx-radio-group>
7373
</article>
7474
</div>

0 commit comments

Comments
 (0)