Skip to content
This repository was archived by the owner on Apr 28, 2025. It is now read-only.

Commit a859c31

Browse files
authored
chore(deprecations): restructed sass code to fix deprecation messages (#2507)
1 parent 364355c commit a859c31

File tree

6 files changed

+29
-28
lines changed

6 files changed

+29
-28
lines changed

src/sass/button/button.scss

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

55
button.btn,
66
a.fake-btn {
7-
@include btn-base();
8-
97
background-color: transparent;
108
border-radius: var(--btn-border-radius, calc($button-height-regular / 2));
119
color: inherit;
@@ -14,6 +12,8 @@ a.fake-btn {
1412
min-height: $button-height-regular;
1513
min-width: 88px;
1614
padding: $button-padding-vertical $button-padding-horizontal;
15+
16+
@include btn-base();
1717
}
1818

1919
button.btn[disabled],
@@ -85,11 +85,11 @@ a.fake-btn--fluid {
8585

8686
.btn__cell,
8787
.fake-btn__cell {
88+
justify-content: center;
89+
8890
@include btn-cell-base();
8991
@include btn-cell-fixed-height();
9092
@include btn-cell-truncated();
91-
92-
justify-content: center;
9393
}
9494

9595
button.btn--form .btn__cell,
@@ -520,12 +520,12 @@ a.fake-btn--truncated {
520520

521521
button.btn--large-truncated,
522522
a.fake-btn--large-truncated {
523-
@include btn-truncate();
524-
525523
font-size: var(--font-size-medium);
526524
height: $button-height-large;
527525
min-height: $button-height-large;
528526
padding: $button-padding-vertical-large $button-padding-horizontal;
527+
528+
@include btn-truncate();
529529
}
530530

531531
button.btn--split-start,

src/sass/carousel/carousel.scss

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,6 @@ $ebay-carousel-transition-function: ease-in-out;
1919
color-background-primary
2020
);
2121

22-
&--prev {
23-
left: calc(var(--spacing-200) * -1);
24-
margin-inline-end: 1px;
25-
}
26-
27-
&--next {
28-
margin-inline-start: 1px;
29-
right: calc(var(--spacing-200) * -1);
30-
}
31-
3222
align-items: center;
3323
border: 1px solid;
3424
border-radius: 16px;
@@ -53,6 +43,16 @@ $ebay-carousel-transition-function: ease-in-out;
5343
color-stroke-subtle
5444
);
5545

46+
&--prev {
47+
left: calc(var(--spacing-200) * -1);
48+
margin-inline-end: 1px;
49+
}
50+
51+
&--next {
52+
margin-inline-start: 1px;
53+
right: calc(var(--spacing-200) * -1);
54+
}
55+
5656
svg {
5757
@include color-token(
5858
carousel-paddle-foreground-color,

src/sass/cta-button/cta-button.scss

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
@import "../mixins/private/token-mixins";
44

55
a.cta-btn {
6-
@include btn-base();
7-
86
@include background-color-token(
97
cta-btn-background-color,
108
color-background-primary
@@ -20,6 +18,7 @@ a.cta-btn {
2018
max-width: 100%;
2119
padding: $button-padding-vertical 20px;
2220
@include color-token(cta-btn-foreground-color, color-foreground-primary);
21+
@include btn-base();
2322
}
2423

2524
a.cta-btn:visited {
@@ -67,10 +66,11 @@ a.cta-btn--fluid {
6766

6867
span.cta-btn__cell {
6968
@include btn-cell-base();
70-
@include btn-cell-fixed-height();
71-
@include btn-cell-truncated();
7269

7370
justify-content: center;
71+
72+
@include btn-cell-fixed-height();
73+
@include btn-cell-truncated();
7474
}
7575

7676
a.cta-btn svg.icon {

src/sass/filter-menu/filter-menu.scss

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,9 @@ span.filter-menu-form__item {
7070

7171
.filter-menu__item[role^="menuitem"],
7272
.filter-menu-form__item {
73-
@include filter-menu-item-base();
74-
7573
padding: 8px 16px;
7674

75+
@include filter-menu-item-base();
7776
&:last-child {
7877
margin-bottom: 8px;
7978
}
@@ -113,14 +112,13 @@ span.filter-menu-form__item {
113112

114113
button.filter-menu__footer,
115114
button.filter-menu-form__footer[type="submit"] {
116-
@include filter-menu-item-base();
117-
118115
border: none;
119116
border-top-style: solid;
120117
border-top-width: 1px;
121118
bottom: 0;
122119
padding: 16px;
123120
@include border-color-token(filter-menu-border-color, color-stroke-subtle);
121+
@include filter-menu-item-base();
124122

125123
&:hover {
126124
background-color: var(--color-state-primary-hover);

src/sass/mixins/private/_button-mixins.scss

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,11 @@ $button-border-radius-large: 24px;
2323
}
2424

2525
&--truncated {
26-
@include btn-truncate();
26+
& {
27+
height: 40px;
28+
}
2729

28-
height: 40px;
30+
@include btn-truncate();
2931
}
3032

3133
&:focus-visible {

src/sass/mixins/private/_menu-mixins.scss

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,18 @@
11
@import "./selection-list-mixins";
22

33
@mixin menu-menuitem-base($component) {
4-
@include selection-list-item-base();
5-
64
@include border-color-token(
75
#{$component}-menuitem-border-color,
86
color-background-primary
97
);
8+
109
@include color-token(
1110
#{$component}-menuitem-foreground-color,
1211
color-foreground-primary
1312
);
1413

14+
@include selection-list-item-base();
15+
1516
&:hover {
1617
background-color: var(--color-state-primary-hover);
1718
@include color-token(

0 commit comments

Comments
 (0)