Skip to content

Commit b9aa549

Browse files
committed
feat: Make Settings icon more consistent with the NavItems
1 parent c224285 commit b9aa549

File tree

2 files changed

+22
-13
lines changed

2 files changed

+22
-13
lines changed

src/layout/Navbar/NavButton.svelte

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,6 @@
8181
}
8282
8383
@media (max-width: 767px) {
84-
li {
85-
border-radius: 100px;
86-
}
8784
a {
8885
padding: 0.75rem 1.25rem;
8986
justify-content: left;

src/layout/Navbar/NavHost.svelte

Lines changed: 22 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,11 @@
9191
</ul>
9292
</div>
9393
<div id="secondary-navigation">
94-
<button on:click={() => (modals.settings = !modals.settings)} aria-label="Settings">
95-
<Cog size="20px" color="var(--surface-six)" />
96-
</button>
94+
<span class:selected={modals.settings}>
95+
<button on:click={() => (modals.settings = !modals.settings)} aria-label="Settings">
96+
<Cog size="20px" color={modals.settings ? 'var(--primary)' : 'var(--surface-six)'} />
97+
</button>
98+
</span>
9799
</div>
98100
</div>
99101
</div>
@@ -119,6 +121,23 @@
119121
<SessionExpiredModal bind:loginOpen={modals.login} />
120122

121123
<style lang="scss">
124+
#secondary-navigation {
125+
display: flex;
126+
127+
span {
128+
border-radius: 10px;
129+
padding: 10px 16px;
130+
131+
&:hover {
132+
background-color: var(--surface-three);
133+
}
134+
135+
&.selected {
136+
background-color: var(--tertiary);
137+
}
138+
}
139+
}
140+
122141
#logo {
123142
padding: 0.5rem;
124143
}
@@ -149,13 +168,6 @@
149168
background-color: var(--surface-eight);
150169
}
151170
152-
#main-navigation,
153-
#secondary-navigation {
154-
align-items: center;
155-
display: flex;
156-
gap: 2rem;
157-
}
158-
159171
img {
160172
height: 22px;
161173
}

0 commit comments

Comments
 (0)