Skip to content

Commit a93381f

Browse files
authored
Additional 4px-based spacing component changes (#7576)
1 parent 4e3009b commit a93381f

File tree

44 files changed

+104
-108
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+104
-108
lines changed

packages/core/src/_reset.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ body {
2525
}
2626

2727
p {
28-
margin-bottom: $pt-spacing * 2.5;
28+
margin-bottom: $pt-spacing * 2;
2929
margin-top: 0;
3030
}
3131

packages/core/src/_typography.scss

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Styleguide headings
2828

2929
.#{$ns}-heading {
3030
@include heading-typography();
31-
margin: 0 0 ($pt-spacing * 2.5);
31+
margin: 0 0 ($pt-spacing * 3);
3232
padding: 0;
3333
}
3434

@@ -155,7 +155,7 @@ Styleguide running-text
155155
}
156156

157157
p {
158-
margin: 0 0 ($pt-spacing * 2.5);
158+
margin: 0 0 ($pt-spacing * 3);
159159
padding: 0;
160160
}
161161

@@ -252,7 +252,7 @@ Styleguide preformatted
252252
display: block;
253253
font-size: $pt-font-size - 1px;
254254
line-height: 1.4;
255-
margin: ($pt-spacing * 2.5) 0;
255+
margin: ($pt-spacing * 2) 0;
256256
padding: ($pt-spacing * 3) ($pt-spacing * 4);
257257
word-break: break-all;
258258
word-wrap: break-word;
@@ -309,7 +309,7 @@ Styleguide blockquote
309309

310310
%blockquote {
311311
border-left: solid $pt-spacing rgba($gray4, 0.5);
312-
margin: 0 0 ($pt-spacing * 2.5);
312+
margin: 0 0 ($pt-spacing * 2);
313313
padding: 0 ($pt-spacing * 5);
314314

315315
.#{$ns}-dark & {
@@ -344,7 +344,7 @@ Styleguide lists
344344
*/
345345

346346
%list {
347-
margin: ($pt-spacing * 2.5) 0;
347+
margin: ($pt-spacing * 2) 0;
348348
padding-left: $pt-spacing * 7.5;
349349

350350
li:not(:last-child) {

packages/core/src/components/alert/_alert.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,9 @@
2525
.#{$ns}-alert-footer {
2626
display: flex;
2727
flex-direction: row-reverse;
28-
margin-top: $pt-spacing * 2.5;
28+
margin-top: $pt-spacing * 3;
2929

3030
.#{$ns}-button {
31-
margin-left: $pt-spacing * 2.5;
31+
margin-left: $pt-spacing * 2;
3232
}
3333
}

packages/core/src/components/button/_common.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@
77
@import "../progress-bar/common";
88

99
$button-border-width: 1px !default;
10-
$button-padding: $pt-spacing ($pt-spacing * 2.5) !default;
10+
$button-padding: $pt-spacing ($pt-spacing * 2) !default;
1111
$button-padding-small: 0 ($pt-spacing * 2) !default;
1212
$button-padding-large: $pt-spacing ($pt-spacing * 4) !default;
1313
$button-icon-spacing: $pt-spacing * 2 !default;
14-
$button-icon-spacing-large: $pt-spacing * 2.5 !default;
14+
$button-icon-spacing-large: $pt-spacing * 2 !default;
1515

1616
/*
1717
CSS `border` property issues:

packages/core/src/components/callout/_callout.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
$callout-padding: $pt-spacing * 4;
77
$callout-header-margin-top: $pt-spacing * 0.5;
8-
$callout-padding-compact: $pt-spacing * 2.5;
8+
$callout-padding-compact: $pt-spacing * 2;
99

1010
.#{$ns}-callout {
1111
@include running-typography();

packages/core/src/components/card/_card-variables.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ $card-list-border-width: 1px !default;
1313

1414
// CardList Card item min-height is calculated as height of a button + vertical padding.
1515
// We need to add an extra pixel to account for the bottom border.
16-
$card-list-item-padding-vertical: $pt-spacing * 2.5 !default;
16+
$card-list-item-padding-vertical: $pt-spacing * 2 !default;
1717
// prettier-ignore
1818
$card-list-item-min-height: $pt-button-height + ($card-list-item-padding-vertical * 2) + $card-list-border-width !default;
1919
$card-list-item-padding: $card-list-item-padding-vertical $card-padding !default;

packages/core/src/components/control-card/_control-card.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
// control indicators should be top-aligned
2222
align-items: flex-start;
2323
display: flex;
24-
gap: $pt-spacing * 2.5;
24+
gap: $pt-spacing * 2;
2525
margin: 0;
2626
padding: $card-padding;
2727
width: calc(100%);

packages/core/src/components/dialog/_dialog-footer.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
gap: $dialog-padding;
1919
justify-content: space-between;
2020
margin: 0;
21-
padding: ($pt-spacing * 2.5) ($pt-spacing * 2.5) ($pt-spacing * 2.5) $dialog-padding;
21+
padding: ($pt-spacing * 2) ($pt-spacing * 2) ($pt-spacing * 2) $dialog-padding;
2222

2323
.#{$ns}-dark & {
2424
background: $dark-gray4;
@@ -35,6 +35,6 @@
3535
justify-content: flex-end;
3636

3737
.#{$ns}-button {
38-
margin-left: $pt-spacing * 2.5;
38+
margin-left: $pt-spacing * 2;
3939
}
4040
}

packages/core/src/components/dialog/_multistep-dialog.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ $step-radius: $pt-border-radius * 2 !default;
205205
.#{$ns}-dialog-step-title {
206206
color: $pt-text-color-disabled;
207207
flex: 1;
208-
padding-left: $pt-spacing * 2.5;
208+
padding-left: $pt-spacing * 2;
209209

210210
.#{$ns}-dark & {
211211
color: $pt-dark-text-color-disabled;

packages/core/src/components/entity-title/_entity-title.scss

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,6 @@
8181
&-heading-h3 {
8282
gap: $pt-spacing * 4;
8383

84-
.#{$ns}-entity-title-status-tag {
85-
margin-left: $pt-spacing * 2.5;
86-
}
87-
8884
.#{$ns}-entity-title-subtitle {
8985
font-size: $pt-font-size;
9086
}
@@ -93,7 +89,7 @@
9389
&-heading-h4,
9490
&-heading-h5,
9591
&-heading-h6 {
96-
gap: $pt-spacing * 2.5;
92+
gap: $pt-spacing * 2;
9793

9894
.#{$ns}-entity-title-subtitle {
9995
font-size: $pt-font-size-small;

0 commit comments

Comments
 (0)