Skip to content

Commit 3193f19

Browse files
committed
Header: Use fluid space scale
1 parent cd72d24 commit 3193f19

File tree

1 file changed

+20
-15
lines changed

1 file changed

+20
-15
lines changed

app/components/header.module.css

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"logo search nav" auto /
1313
auto 1fr auto;
1414
align-items: center;
15-
padding: 10px 20px;
15+
padding: var(--space-2xs) var(--space-m);
1616
color: white;
1717

1818
a {
@@ -54,24 +54,25 @@
5454
align-items: center;
5555

5656
h1 {
57-
font-size: 24px;
57+
margin: 0;
58+
font-size: var(--space-m);
5859
}
5960
}
6061

6162
.logo {
62-
width: 60px;
63+
width: var(--space-xl);
6364
height: auto;
6465
/* negative margin to account for blank space in the image */
6566
margin-left: -10px;
66-
margin-right: 10px;
67+
margin-right: calc(var(--space-s) - 10px);
6768
}
6869

6970
.search-form {
7071
grid-area: search;
71-
padding: 0 16px;
72+
margin: 0 var(--space-m);
7273

7374
@media only screen and (max-width: 820px) {
74-
padding: 10px 0;
75+
margin: var(--space-s) 0;
7576
}
7677

7778
.hero & {
@@ -102,7 +103,7 @@
102103
}
103104

104105
.sep {
105-
margin: 0 10px;
106+
margin: 0 var(--space-2xs);
106107
opacity: 0.5;
107108
}
108109

@@ -135,27 +136,31 @@
135136
composes: button-reset from '../styles/shared/buttons.module.css';
136137
display: inline-flex;
137138
align-items: center;
138-
margin: 0 -5px;
139-
padding: 5px;
139+
/* negative margin for larger click target */
140+
margin: calc(var(--space-2xs) * -1);
141+
padding: var(--space-2xs);
140142
cursor: pointer;
141143

142144
&:disabled {
143145
cursor: wait;
144146
}
145147

146148
.spinner {
147-
margin-right: 6px;
149+
--spinner-color: white;
150+
--spinner-bg-color: rgba(255, 255, 255, .2);
151+
152+
margin-right: var(--space-2xs);
148153
}
149154
}
150155

151156
.login-icon {
152-
width: 16px;
153-
margin-right: 6px;
157+
width: 1em;
158+
margin-right: var(--space-2xs);
154159
opacity: 0.5;
155160
}
156161

157162
.avatar {
158-
margin-right: 0.5em;
163+
margin-right: var(--space-2xs);
159164
}
160165

161166
.current-user-links {
@@ -167,7 +172,7 @@
167172
.dropdown-button {
168173
background: none;
169174
border: 0;
170-
padding: 10px 0;
175+
padding: 0;
171176

172177
img {
173178
vertical-align: top;
@@ -184,6 +189,6 @@
184189
}
185190

186191
.spinner {
187-
margin-right: 6px;
192+
margin-right: var(--space-2xs);
188193
}
189194
}

0 commit comments

Comments
 (0)