Skip to content

Commit 47e63ab

Browse files
authored
refactor(list): item gaps and margins (#15066)
1 parent 6aff96c commit 47e63ab

File tree

1 file changed

+7
-53
lines changed

1 file changed

+7
-53
lines changed

projects/igniteui-angular/src/lib/core/styles/components/list/_list-theme.scss

Lines changed: 7 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -287,30 +287,6 @@
287287
'indigo': rem(8px),
288288
), $variant);
289289

290-
$list-icon-margin-inline: (
291-
comfortable: rem(16px),
292-
cosy: rem(8px),
293-
compact: rem(4px)
294-
);
295-
296-
$list-icon-margin-block: (
297-
comfortable: rem(8px),
298-
cosy: rem(6px),
299-
compact: rem(4px)
300-
);
301-
302-
$list-line-margin: (
303-
comfortable: rem(16px),
304-
cosy: rem(12px),
305-
compact: rem(8px)
306-
);
307-
308-
$list-actions-margin: (
309-
comfortable: rem(8px),
310-
cosy: rem(6px),
311-
compact: rem(4px)
312-
);
313-
314290
%igx-list {
315291
@include sizable();
316292

@@ -326,6 +302,10 @@
326302
z-index: 0;
327303
border-radius: var-get($theme, 'border-radius');
328304

305+
@if $variant == 'bootstrap' {
306+
border: rem(1px) solid var-get($theme, 'border-color');
307+
}
308+
329309
&:focus-visible {
330310
outline-style: none;
331311
}
@@ -479,21 +459,14 @@
479459
align-items: center;
480460
justify-content: center;
481461
color: var-get($theme, 'item-action-color');
482-
483-
@if $variant == 'indigo' {
484-
gap: sizable(rem(4), rem(6), rem(8));
485-
}
462+
gap: if($variant == 'indigo', sizable(rem(4), rem(6), rem(8)), rem(8px));
486463

487464
&:empty {
488465
display: none;
489466
}
490467

491468
> * {
492469
--component-size: #{if($variant == 'indigo', 2, var(--list-size))};
493-
494-
@if $variant != 'indigo' {
495-
margin-inline-start: pad-inline(map.get($list-actions-margin, 'compact'), map.get($list-actions-margin, 'cosy'), map.get($list-actions-margin, 'comfortable'));
496-
}
497470
}
498471

499472
igc-icon,
@@ -517,10 +490,7 @@
517490
border-radius: var-get($theme, 'item-border-radius');
518491
background: inherit;
519492
z-index: 2;
520-
521-
@if $variant == 'indigo' {
522-
gap: rem(8px);
523-
}
493+
gap: if($variant == 'indigo', rem(8px), rem(16px));
524494
}
525495

526496
%igx-list-header,
@@ -533,7 +503,7 @@
533503
display: flex;
534504
align-items: center;
535505
justify-content: center;
536-
align-self: if($variant == 'indigo', center, flex-start);
506+
align-self: center;
537507
padding: 0;
538508
color: var-get($theme, 'item-thumbnail-color');
539509
gap: rem(8px);
@@ -542,12 +512,6 @@
542512
> igx-icon,
543513
> igc-icon {
544514
--component-size: #{if($variant == 'indigo', 2, var(--list-size))};
545-
546-
@if $variant != 'indigo' {
547-
margin-inline-end: pad-inline(map.get($list-icon-margin-inline, 'compact'), map.get($list-icon-margin-inline, 'cosy'), map.get($list-icon-margin-inline, 'comfortable'));
548-
margin-inline-start: 0;
549-
margin-block: pad-block(map.get($list-icon-margin-block, 'compact'), map.get($list-icon-margin-block, 'cosy'), map.get($list-icon-margin-block, 'comfortable'));
550-
}
551515
}
552516

553517
&:empty {
@@ -557,16 +521,6 @@
557521

558522
%igx-list__item-thumbnail:not(:empty) + %igx-list__item-lines {
559523
--component-size: var(--list-size);
560-
561-
@if $variant != 'indigo' {
562-
margin-inline-start: pad-inline(map.get($list-line-margin, 'compact'), map.get($list-line-margin, 'cosy'), map.get($list-line-margin, 'comfortable'));
563-
}
564-
}
565-
566-
@if $variant != 'indigo' {
567-
%igx-list__item-lines + %igx-list__item-actions {
568-
margin-inline-start: rem(8px);
569-
}
570524
}
571525

572526
%igx-list-item-content--active {

0 commit comments

Comments
 (0)