Skip to content

Commit e633a2d

Browse files
committed
update components to use em spacer variables:
- flash - breadcrumb - button - states - markdown lists - markdown-body - support button mixins - tooltips
1 parent 1613139 commit e633a2d

File tree

8 files changed

+10
-10
lines changed

8 files changed

+10
-10
lines changed

modules/primer-alerts/lib/flash.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373
}
7474

7575
.warning {
76-
padding: 0.5em;
76+
padding: $em-spacer-5;
7777
margin-bottom: 0.8em;
7878
font-weight: $font-weight-bold;
7979
background-color: $yellow-100;

modules/primer-breadcrumb/lib/breadcrumb.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
list-style: none;
66

77
&::after {
8-
padding-right: 0.5em;
9-
padding-left: 0.5em;
8+
padding-right: $em-spacer-5;
9+
padding-left: $em-spacer-5;
1010
color: $border-gray;
1111
content: "/";
1212
}

modules/primer-buttons/lib/button.scss

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

9191
// Large button adds more padding around text. Use font-size utils to increase font-size.. e.g, <p class="text-gamma"><button class="btn btn-large btn-primary" type="button">Big green button</button></p>
9292
.btn-large {
93-
padding: 0.75em 1.25em;
93+
padding: $em-spacer-6 1.25em;
9494
font-size: inherit;
9595
border-radius: 6px;
9696
}

modules/primer-labels/lib/states.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
.State--red { background-color: $red-600; }
2727

2828
.State--small {
29-
padding: 0.125em $spacer-1;
29+
padding: $em-spacer-2 $spacer-1;
3030
font-size: $h6-size;
3131

3232
.octicon {

modules/primer-markdown/lib/lists.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
}
5555

5656
li + li {
57-
margin-top: 0.25em;
57+
margin-top: $em-spacer-3;
5858
}
5959

6060
dl {

modules/primer-markdown/lib/markdown-body.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@
6969
}
7070

7171
hr {
72-
height: 0.25em;
72+
height: $em-spacer-3;
7373
padding: 0;
7474
margin: $spacer-4 0;
7575
background-color: $gray-200;

modules/primer-support/lib/mixins/buttons.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
&.hover {
1919
background-color: darken($bg2, 3%);
2020
background-image: linear-gradient(-180deg, darken($bg, 3%) 0%, darken($bg2, 3%) 90%);
21-
background-position: 0 -0.5em;
21+
background-position: 0 -$em-spacer-5;
2222
border-color: rgba($black, 0.35);
2323
}
2424

@@ -50,7 +50,7 @@
5050
&.hover {
5151
background-color: darken($bg2, 2%);
5252
background-image: linear-gradient(-180deg, darken($bg, 2%) 0%, darken($bg2, 2%) 90%);
53-
background-position: 0 -0.5em;
53+
background-position: 0 -$em-spacer-5;
5454
border-color: $black-fade-50;
5555
}
5656

modules/primer-tooltips/lib/tooltips.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
position: absolute;
88
z-index: 1000000;
99
display: none;
10-
padding: 0.5em 0.75em;
10+
padding: $em-spacer-5 $em-spacer-6;
1111
font: normal normal 11px/1.5 $body-font;
1212
-webkit-font-smoothing: subpixel-antialiased;
1313
color: $tooltip-text-color;

0 commit comments

Comments
 (0)