Skip to content

Commit 23da641

Browse files
Refresh: Standardize border widths (mozilla#15435)
1 parent 3a16d18 commit 23da641

7 files changed

Lines changed: 19 additions & 16 deletions

File tree

media/css/m24/components/footer-newsletter.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
$max-footer-content-width: $content-max;
66

77
@mixin divider-line {
8-
border-bottom: 4px solid $m24-color-light-gray;
8+
border-bottom: $border-width solid $m24-color-light-gray;
99
}
1010

1111
// hide details for JS users
@@ -143,7 +143,7 @@ $max-footer-content-width: $content-max;
143143

144144
input[type="email"],
145145
select {
146-
border: 2px solid $m24-color-black;
146+
border: $border-width solid $m24-color-black;
147147
border-radius: 0;
148148
background-color: $m24-color-light-gray;
149149
color: $m24-color-black;
@@ -248,7 +248,7 @@ $max-footer-content-width: $content-max;
248248
margin-bottom: 0;
249249

250250
button {
251-
border: 2px solid $m24-color-green;
251+
border: $border-width solid $m24-color-green;
252252
border-radius: 0;
253253
font-family: $secondary-font;
254254
position: relative;

media/css/m24/components/footer-refresh.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
$max-footer-content-width: $content-max;
66

77
@mixin divider-line {
8-
border-bottom: 4px solid $m24-color-light-gray;
8+
border-bottom: $border-width solid $m24-color-light-gray;
99
}
1010

1111
// whole footer
@@ -236,7 +236,7 @@ $max-footer-content-width: $content-max;
236236
background-color: transparent;
237237
color: $m24-color-black;
238238
padding: 6px 24px;
239-
border: 2px solid $m24-color-green;
239+
border: $border-width solid $m24-color-green;
240240
text-decoration: none;
241241
text-align: center;
242242
max-width: 800px;
@@ -334,7 +334,7 @@ $max-footer-content-width: $content-max;
334334
}
335335

336336
.mzp-js-language-switcher-select {
337-
border: 2px solid transparent;
337+
border: $border-width solid transparent;
338338
border-radius: 0;
339339
font-family: $secondary-font;
340340
font-weight: 600;

media/css/m24/components/navigation-refresh.scss

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424

2525
.m24-navigation-refresh {
2626
background-color: $color-white;
27-
border-bottom: 2px solid $m24-color-medium-gray;
27+
border-bottom: $border-width solid $m24-color-medium-gray;
2828
display: flex;
2929
width: 100%;
3030

@@ -47,7 +47,7 @@
4747
&.mzp-is-scrolling {
4848
// Shadow colors are equivalent to $color-ink-90, $color-blue-90, $color-ink-90
4949
// We can't use a $box-shadow token here because it needs a different size and offset
50-
box-shadow: 0 0 6px 1px rgba(29, 17, 51, 0.04), 0 0 8px 2px rgba(9, 32, 77, 0.12), 0 0 5px -3px rgba(29, 17, 51, 0.12);
50+
box-shadow: 0 0 6px 1px rgba(29, 17, 51, 0.04), 0 0 8px $border-width rgba(9, 32, 77, 0.12), 0 0 5px -3px rgba(29, 17, 51, 0.12);
5151
}
5252

5353
&.mzp-is-hidden {
@@ -184,7 +184,7 @@
184184
}
185185

186186
.m24-c-navigation-menu {
187-
border-top: 2px solid $m24-color-medium-gray;
187+
border-top: $border-width solid $m24-color-medium-gray;
188188
margin-bottom: 0;
189189
width: 100%;
190190

@@ -292,7 +292,7 @@
292292
}
293293

294294
.m24-c-menu-category {
295-
border-bottom: 2px solid $token-color-light-gray;
295+
border-bottom: $border-width solid $token-color-light-gray;
296296
border-top: none;
297297
padding: 8px 16px;
298298
position: relative;
@@ -355,7 +355,7 @@
355355
@include border-box;
356356

357357
@media #{$mq-md} {
358-
border-bottom: 4px solid $m24-color-medium-gray;
358+
border-bottom: $border-width solid $m24-color-medium-gray;
359359
left: 0;
360360
overflow: auto;
361361
padding: 0;
@@ -468,11 +468,11 @@
468468
}
469469

470470
& > li {
471-
border-bottom: 2px solid transparent;
471+
border-bottom: $border-width solid transparent;
472472
width: 100%;
473473

474474
@media #{$mq-md} {
475-
border-bottom: 2px solid $token-color-light-gray;
475+
border-bottom: $border-width solid $token-color-light-gray;
476476
}
477477

478478
.m24-c-menu-item {

media/css/m24/components/pencil-banner.scss

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

55
.m24-pencil-banner {
66
background-color: $m24-color-light-green;
7-
border-bottom: 2px solid $m24-color-green;
7+
border-bottom: $border-width solid $m24-color-green;
88
padding: $spacer-sm;
99

1010
.m24-pencil-banner-copy {

media/css/m24/grid.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@
8181
color: inherit;
8282

8383
.m24-c-grid-tile-image > * {
84-
outline: 2px solid transparent;
84+
outline: $border-width solid transparent;
8585
transition: outline-color 150ms ease-in-out;
8686
}
8787

media/css/m24/spring.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ $logo-spacing: calc(24px + #{$spacer-xs});
1010
}
1111

1212
.m24-c-spring-item {
13-
border-bottom: 4px solid $m24-color-light-gray;
13+
border-bottom: $border-width solid $m24-color-light-gray;
1414
margin-bottom: 0;
1515
}
1616

media/css/m24/vars/_spacing.scss

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,3 +14,6 @@ $spacer-2xs: var(--spacer-2xs);
1414

1515
// container
1616
$container-padding: var(--container-padding);
17+
18+
// border
19+
$border-width: 2px;

0 commit comments

Comments
 (0)