Skip to content

Commit b0ba0df

Browse files
committedMay 21, 2024
Use px instead of rem for borders
1 parent 5dbbcad commit b0ba0df

File tree

9 files changed

+48
-41
lines changed

9 files changed

+48
-41
lines changed
 

‎.stylelintrc.yml

+9-5
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ extends:
44

55
plugins:
66
# TODO: stylelint-a11y
7-
# https://github.com/developer-stylechain/stylelint-a11y
7+
# https://github.com/developer-stylechain/stylelint-a11y
88
- stylelint-plugin-defensive-css
99

1010
# Default rules
@@ -121,18 +121,22 @@ rules:
121121
# Units blacklist, with exceptions
122122
unit-disallowed-list:
123123
- [ms, pt, px]
124-
- ignoreProperties:
124+
- ignoreFunctions:
125+
- px2em
126+
- px2rem
127+
ignoreProperties:
125128
px:
129+
- border
130+
- border-bottom
131+
- border-radius
132+
- border-top
126133
- bottom
127134
- box-shadow
128135
- font-size
129136
- left
130137
- right
131138
- text-shadow
132139
- top
133-
ignoreFunctions:
134-
- px2em
135-
- px2rem
136140

137141
# Lowercase keywords, with exceptions
138142
value-keyword-case:

‎_sass/base/_base.scss

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ address {
7676
hr {
7777
margin: 1rem 0;
7878
border: 0;
79-
border-top: px2rem(1px) solid var(--brand-color);
79+
border-top: 1px solid var(--brand-color);
8080
box-shadow: 0 0 15px var(--brand-color--shadow);
8181
}
8282

‎_sass/components/_terminal.scss

+7-5
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
.terminal {
2-
$border-radius: px2rem(5px);
3-
$scrollbar-w: px2rem(10px);
2+
// stylelint-disable unit-disallowed-list
3+
$border-radius: 5px;
4+
$scrollbar-w: 10px;
5+
// stylelint-enable unit-disallowed-list
46
width: px2rem(650px);
57
margin: 0 auto 1rem;
68
font-size: 0.875rem;
7-
border: px2rem(1px) solid var(--border-color);
8-
border-radius: calc(#{$border-radius} + #{px2rem(1px)});
9+
border: 1px solid var(--border-color);
10+
border-radius: calc(#{$border-radius} + 1px);
911
scrollbar-width: #{$scrollbar-w};
1012
@include respond-to('large') {
1113
width: 100%;
@@ -22,7 +24,7 @@
2224
padding: px2rem(2px) px2rem(6px);
2325
line-height: 1.2;
2426
background: linear-gradient(180deg, #{var(--gradient-color--from)} 25%, #{var(--gradient-color--to)} 75%);
25-
border-bottom: px2rem(1px) solid var(--border-color);
27+
border-bottom: 1px solid var(--border-color);
2628
border-top-left-radius: #{$border-radius};
2729
border-top-right-radius: #{$border-radius};
2830
.task {

‎_sass/components/admonitions/_index.scss

+4-3
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
.admonition {
2-
$border-radius: px2rem(6px);
2+
// stylelint-disable-next-line unit-disallowed-list
3+
$border-radius: 6px;
34
$padding: 0.66rem;
45
max-width: px2rem(400px);
56
margin: 0 auto 1rem;
67
background-color: var(--bg-color--clearest);
7-
border: px2rem(2px) solid var(--text-color);
8-
border-radius: calc(#{$border-radius} + #{px2rem(1px)});
8+
border: 2px solid var(--text-color);
9+
border-radius: calc(#{$border-radius} + 1px);
910
.header,
1011
.footer {
1112
background-color: var(--bg-color--light);

‎_sass/components/cards/_machine.scss

+4-4
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
// Project info
1111
&__machine {
1212
font-size: 1rem;
13-
border: px2rem(2px) solid var(--text-color);
14-
border-radius: px2rem(5px);
13+
border: 2px solid var(--text-color);
14+
border-radius: 5px;
1515
.title {
1616
margin-bottom: 0.5rem;
1717
font-size: 1.25rem;
@@ -70,14 +70,14 @@
7070
}
7171
}
7272
hr {
73-
border-top: px2rem(1px) solid var(--text-color);
73+
border-top: 1px solid var(--text-color);
7474
box-shadow: 0 0 15px var(--text-color--shadow);
7575
}
7676
}
7777
// Blog post
7878
&__machine-alt {
7979
padding: px2rem(6px) px2rem(12px);
80-
border: px2rem(10px) solid var(--border-color--dark);
80+
border: 10px solid var(--border-color--dark);
8181
.date {
8282
font-size: 0.825rem;
8383
color: var(--text-color--secondary);

‎_sass/components/cards/_samaritan.scss

+10-10
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,8 @@
103103
width: px2em(66px, $font-size-small);
104104
}
105105
.content {
106-
border-top: px2em(2px, $font-size-small) solid var(--bg-color--lightest);
107-
border-bottom: px2em(2px, $font-size-small) solid var(--bg-color--lightest);
106+
border-top: 2px solid var(--bg-color--lightest);
107+
border-bottom: 2px solid var(--bg-color--lightest);
108108
&__row {
109109
flex-flow: row nowrap;
110110
margin: px2em(5px, $font-size-small) px2em(10px, $font-size-small);
@@ -119,7 +119,7 @@
119119
}
120120
// Header, used in top artists page
121121
&__samaritan-header {
122-
border: px2rem(2px) solid var(--text-color);
122+
border: 2px solid var(--text-color);
123123
.title {
124124
display: flex;
125125
flex-flow: row nowrap;
@@ -163,7 +163,7 @@
163163
width: 100%;
164164
margin-top: px2rem(2px);
165165
content: "";
166-
border-bottom: px2rem(2px) solid var(--border-color--dark);
166+
border-bottom: 2px solid var(--border-color--dark);
167167
}
168168
// Card title
169169
/* stylelint-disable-next-line no-descending-specificity */
@@ -179,14 +179,14 @@
179179
flex-wrap: nowrap;
180180
padding: px2rem(6px);
181181
background-color: var(--bg-color--light);
182-
border-top: px2rem(1px) solid var(--border-color--dark);
183-
border-bottom: px2rem(1px) solid var(--border-color--dark);
182+
border-top: 1px solid var(--border-color--dark);
183+
border-bottom: 1px solid var(--border-color--dark);
184184
.image {
185185
padding: px2rem(4px);
186186
margin-right: px2rem(3px);
187187
background-color: var(--bg-color);
188-
border: px2rem(1px) solid var(--border-color--clear);
189-
border-radius: px2rem(4px);
188+
border: 1px solid var(--border-color--clear);
189+
border-radius: 4px;
190190
@include respond-to('smaller') {
191191
display: none;
192192
}
@@ -201,7 +201,7 @@
201201
width: 100%;
202202
padding: 0.75em;
203203
margin-left: px2rem(3px);
204-
border: px2rem(1px) solid var(--border-color--clear);
204+
border: 1px solid var(--border-color--clear);
205205
@include respond-to('smaller') {
206206
border: 0;
207207
}
@@ -228,7 +228,7 @@
228228
.container__list {
229229
hr {
230230
margin: 0.1em 0;
231-
border-top: px2rem(1px) solid var(--border-color--clear);
231+
border-top: 1px solid var(--border-color--clear);
232232
box-shadow: unset;
233233
}
234234
.title {

‎_sass/layout/_header.scss

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ header {
1313
// margin-bottom: 3rem;
1414
font-size: $font-size-small;
1515
background-color: var(--bg-color--clearer);
16-
border-bottom: px2em(1px, $font-size-small) solid var(--text-color);
16+
border-bottom: 1px solid var(--text-color);
1717
.logo {
1818
font-family: 'Acens', $font-stack-text-system;
1919
@include respond-to('small') {

‎_sass/pages/_404.scss

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ body {
1313
padding-bottom: px2rem(2px);
1414
font-size: px2rem(26px);
1515
text-decoration: none;
16-
border-bottom: px2rem(2px) solid currentColor;
16+
border-bottom: 2px solid currentColor;
1717
}
1818
img {
1919
width: px2rem(32px);

‎_sass/pages/_blog.scss

+11-11
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ article {
1212
video {
1313
max-width: 100%;
1414
height: auto;
15-
border: px2rem(1px) solid var(--bg-color--lighter);
15+
border: 1px solid var(--bg-color--lighter);
1616
}
1717
figure {
1818
// Liquid tag {% highlight :lang %} puts code blocks inside a <figure> HTML tag
@@ -121,7 +121,7 @@ article {
121121
font-size: 1.15em;
122122
appearance: none;
123123
background-color: var(--table-bg-color--even);
124-
border: px2rem(1px) solid var(--border-color--dark);
124+
border: 1px solid var(--border-color--dark);
125125
&:checked {
126126
&::before {
127127
position: absolute;
@@ -163,7 +163,7 @@ article {
163163
overflow: auto;
164164
overscroll-behavior: contain;
165165
line-height: 1.2;
166-
border: px2rem(1px) solid var(--border-color--dark);
166+
border: 1px solid var(--border-color--dark);
167167
border-radius: 0;
168168
code {
169169
word-break: normal;
@@ -177,7 +177,7 @@ article {
177177
color: var(--text-color--secondary);
178178
word-wrap: break-word;
179179
background-color: var(--bg-color--light);
180-
border: px2rem(1px) solid var(--border-color--dark);
180+
border: 1px solid var(--border-color--dark);
181181
}
182182
}
183183

@@ -199,7 +199,7 @@ article {
199199
text-transform: uppercase;
200200
}
201201
thead {
202-
border-bottom: px2rem(1px) solid var(--border-color--dark);
202+
border-bottom: 1px solid var(--border-color--dark);
203203
}
204204
tbody {
205205
tr {
@@ -212,7 +212,7 @@ article {
212212
}
213213
}
214214
tfoot {
215-
border-top: px2rem(1px) solid var(--border-color--dark);
215+
border-top: 1px solid var(--border-color--dark);
216216
}
217217
thead,
218218
tfoot {
@@ -233,7 +233,7 @@ article {
233233
margin: 1rem;
234234
color: var(--text-color--secondary);
235235
background-color: var(--bg-color--clear);
236-
border: px2rem(8px) solid var(--border-color--dark);
236+
border: 8px solid var(--border-color--dark);
237237
// stylelint-disable-next-line no-descending-specificity
238238
p {
239239
margin-bottom: 0;
@@ -245,7 +245,7 @@ article {
245245
&[title] {
246246
text-decoration: none;
247247
cursor: help;
248-
border-bottom: px2rem(1px) solid var(--brand-color);
248+
border-bottom: 1px solid var(--brand-color);
249249
}
250250
}
251251

@@ -275,7 +275,7 @@ article {
275275
padding: px2rem(6px);
276276
text-transform: uppercase;
277277
user-select: none;
278-
border-bottom: px2rem(2px) solid;
278+
border-bottom: 2px solid;
279279
transition: 0.3s;
280280
& ~ div {
281281
display: flex;
@@ -284,7 +284,7 @@ article {
284284
padding: 0.75rem;
285285
overflow: auto;
286286
overscroll-behavior: contain;
287-
border-bottom: px2rem(2px) solid var(--border-color--dark);
287+
border-bottom: 2px solid var(--border-color--dark);
288288
p {
289289
&:last-of-type {
290290
margin-bottom: 0;
@@ -331,7 +331,7 @@ article {
331331
}
332332
.footnotes {
333333
padding-top: 1rem;
334-
border-top: px2rem(1px) solid var(--brand-color);
334+
border-top: 1px solid var(--brand-color);
335335
// stylelint-disable-next-line no-descending-specificity
336336
ol {
337337
margin: 0;

0 commit comments

Comments
 (0)
Please sign in to comment.