Skip to content

Commit 10a07c4

Browse files
authored
fix(scss notation): fix & notation for utilities / vertical list (#5010)
1 parent ca946b9 commit 10a07c4

File tree

2 files changed

+83
-85
lines changed

2 files changed

+83
-85
lines changed

ui/utilities/vertical-list/_deprecate.scss

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -22,21 +22,21 @@
2222
margin-top: $spacing-x-small;
2323
}
2424
}
25+
}
2526

26-
&_space,
27-
&--space {
27+
.slds-has-cards_space,
28+
.slds-has-cards--space {
2829

29-
> .slds-list__item {
30-
@include border(around, $border-radius-medium);
31-
padding: $spacing-small;
30+
> .slds-list__item {
31+
@include border(around, $border-radius-medium);
32+
padding: $spacing-small;
3233

33-
@include mq-small-min {
34-
padding: $spacing-x-small;
35-
}
34+
@include mq-small-min {
35+
padding: $spacing-x-small;
36+
}
3637

37-
+ .slds-list__item {
38-
margin-top: $spacing-x-small;
39-
}
38+
+ .slds-list__item {
39+
margin-top: $spacing-x-small;
4040
}
4141
}
4242
}

ui/utilities/vertical-list/_index.scss

Lines changed: 72 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -13,23 +13,23 @@
1313
.slds-is-nested {
1414
margin-left: $spacing-medium;
1515
}
16+
}
1617

17-
&_space,
18-
&--space {
18+
.slds-has-block-links_space,
19+
.slds-has-block-links--space {
1920

20-
.slds-list__item,
21-
.slds-item {
22-
padding: 0; // Reset padding if used in conjunction with .slds-has-dividers--{position}-space
23-
}
21+
.slds-list__item,
22+
.slds-item {
23+
padding: 0; // Reset padding if used in conjunction with .slds-has-dividers--{position}-space
24+
}
2425

25-
a {
26-
display: block;
27-
text-decoration: none;
28-
padding: $spacing-small;
26+
a {
27+
display: block;
28+
text-decoration: none;
29+
padding: $spacing-small;
2930

30-
@include mq-medium-min {
31-
padding: $spacing-x-small; // Re-apply that padding to the link
32-
}
31+
@include mq-medium-min {
32+
padding: $spacing-x-small; // Re-apply that padding to the link
3333
}
3434
}
3535
}
@@ -40,16 +40,16 @@
4040
display: inline-block;
4141
text-decoration: none;
4242
}
43+
}
4344

44-
&_space a,
45-
&--space a {
46-
display: inline-block;
47-
text-decoration: none;
48-
padding: $spacing-small;
45+
.slds-has-inline-block-links_space a,
46+
.slds-has-inline-block-links--space a {
47+
display: inline-block;
48+
text-decoration: none;
49+
padding: $spacing-small;
4950

50-
@include mq-medium-min {
51-
padding: $spacing-x-small;
52-
}
51+
@include mq-medium-min {
52+
padding: $spacing-x-small;
5353
}
5454
}
5555

@@ -228,64 +228,62 @@
228228
}
229229
}
230230

231-
.slds-has-divider {
232-
/**
233-
* @summary Adds 1px border divider above an HTML element
234-
*
235-
* @selector .slds-has-divider_top
236-
* @restrict li
237-
* @modifier
238-
*/
239-
&_top,
240-
&--top {
241-
@include border(top);
231+
/**
232+
* @summary Adds 1px border divider above an HTML element
233+
*
234+
* @selector .slds-has-divider_top
235+
* @restrict li
236+
* @modifier
237+
*/
238+
.slds-has-divider_top,
239+
.slds-has-divider--top {
240+
@include border(top);
242241

243-
&-space {
244-
@include border(top);
245-
margin-top: $spacing-x-small;
246-
padding-top: $spacing-x-small;
247-
}
242+
&-space {
243+
@include border(top);
244+
margin-top: $spacing-x-small;
245+
padding-top: $spacing-x-small;
248246
}
247+
}
249248

250-
/**
251-
* @summary Adds dot separator to the right of an HTML element
252-
*
253-
* @selector .slds-has-divider_right
254-
* @restrict li
255-
* @modifier
256-
*/
257-
&_right,
258-
&--right {
259-
@include dot-separator(right);
260-
}
249+
/**
250+
* @summary Adds dot separator to the right of an HTML element
251+
*
252+
* @selector .slds-has-divider_right
253+
* @restrict li
254+
* @modifier
255+
*/
256+
.slds-has-divider_right,
257+
.slds-has-divider--right {
258+
@include dot-separator(right);
259+
}
261260

262-
/**
263-
* @summary Adds 1px border divider below an HTML element
264-
*
265-
* @selector .slds-has-divider_bottom
266-
* @restrict li
267-
* @modifier
268-
*/
269-
&_bottom,
270-
&--bottom {
271-
@include border(bottom);
261+
/**
262+
* @summary Adds 1px border divider below an HTML element
263+
*
264+
* @selector .slds-has-divider_bottom
265+
* @restrict li
266+
* @modifier
267+
*/
268+
.slds-has-divider_bottom,
269+
.slds-has-divider--bottom {
270+
@include border(bottom);
272271

273-
&-space {
274-
@include border(bottom);
275-
margin-bottom: $spacing-x-small;
276-
padding-bottom: $spacing-x-small;
277-
}
272+
&-space {
273+
@include border(bottom);
274+
margin-bottom: $spacing-x-small;
275+
padding-bottom: $spacing-x-small;
278276
}
277+
}
279278

280-
/**
281-
* @summary Adds dot separator to the left of an HTML element
282-
*
283-
* @selector .slds-has-divider_left
284-
* @restrict li
285-
* @modifier
286-
*/
287-
&_left,
288-
&--left {
289-
@include dot-separator(left);
290-
}
279+
/**
280+
* @summary Adds dot separator to the left of an HTML element
281+
*
282+
* @selector .slds-has-divider_left
283+
* @restrict li
284+
* @modifier
285+
*/
286+
.slds-has-divider_left,
287+
.slds-has-divider--left {
288+
@include dot-separator(left);
291289
}

0 commit comments

Comments
 (0)