Skip to content

Commit 5818e98

Browse files
committed
Fixes
1 parent 620dac2 commit 5818e98

File tree

9 files changed

+24
-33
lines changed

9 files changed

+24
-33
lines changed

src/assets/sass/designpatterns.scss

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,7 @@ $colors: (
5151
"a": var(--clr-a),
5252
"a-interact": var(--clr-a-interact),
5353
"focus": var(--clr-focus),
54-
"nav-hover": var(--clr-nav-hover),
55-
"nav-hover-selected": var(--clr-nav-hover-selected),
54+
"nav-interact": var(--clr-nav-interact),
5655
"nav-active": var(--clr-nav-active),
5756
"bkgd": var(--clr-bkgd),
5857
"bkgd-box": var(--clr-bkgd-box),

src/assets/sass/modules/_buttons.scss

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,14 +61,12 @@ button,
6161

6262
&:hover {
6363
filter: brightness(1.05);
64-
text-decoration: none;
6564
}
6665

6766
&:active {
6867
top: 2px;
6968
box-shadow: none;
7069
filter: brightness(0.95);
71-
text-decoration: none;
7270
}
7371

7472
&:focus {

src/assets/sass/modules/_fonts.scss

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
@use "../setup";
22

3-
/* ==========================================================================
4-
Fonts
3+
/* Fonts
54
========================================================================== */
65

76
/**

src/assets/sass/modules/_forms.scss

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -274,20 +274,16 @@ textarea {
274274
}
275275

276276
/**
277-
* Remove the inner padding in Chrome and Safari on macOS.
277+
* Styling for file inputs.
278278
*/
279279

280-
[type="search"]::-webkit-search-decoration {
281-
/* autoprefixer: off */
282-
appearance: none;
283-
}
284-
285280
[type="file"] {
286281
box-sizing: border-box;
287282
height: auto;
288283
min-height: 2em; // 32px / 16px
289284
padding: 0.25em 0.1875em; // 8px / 16px + 3px / 16px
290285
border: 1px solid var(--clr-brdr);
286+
border-radius: 0;
291287
background: var(--clr-bkgd-form);
292288
appearance: none;
293289
cursor: pointer;
@@ -337,6 +333,15 @@ textarea {
337333
filter: brightness(0.95);
338334
}
339335

336+
/**
337+
* Remove the inner padding in Chrome and Safari on macOS.
338+
*/
339+
340+
[type="search"]::-webkit-search-decoration {
341+
/* autoprefixer: off */
342+
appearance: none;
343+
}
344+
340345
/**
341346
* Use indicator icon to signify the drop-down ability of `select`.
342347
*/

src/assets/sass/modules/_links.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ h6 {
111111
padding: 0.25em 0.5em;
112112
transform: translateY(-5em);
113113
transition: transform 0.25s ease-in-out;
114-
background-color: var(--clr-a-alt-interact);
114+
background-color: var(--clr-nav-active);
115115
color: var(--clr-text);
116116

117117
&:focus,

src/assets/sass/modules/_navigation.scss

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -52,32 +52,27 @@
5252
font-weight: bold;
5353

5454
&:hover {
55-
background-color: var(--clr-nav-hover);
55+
background-color: var(--clr-nav-interact);
5656
color: var(--clr-text);
5757
text-decoration: none;
5858
}
5959

6060
&:active {
61-
background-color: var(--clr-nav-active);
62-
color: var(--clr-text-promoted);
61+
filter: brightness(0.95);
6362
text-decoration: none;
6463
}
6564

6665
&:focus {
67-
background-color: var(--clr-nav-hover);
66+
background-color: var(--clr-nav-interact);
6867
}
6968
}
7069

7170
.active a {
72-
background-color: var(--clr-nav-hover);
71+
background-color: var(--clr-nav-active);
7372

7473
&:hover,
7574
&:focus {
76-
background-color: var(--clr-nav-hover-selected);
77-
}
78-
79-
&:active {
80-
background-color: var(--clr-nav-active);
75+
background-color: var(--clr-nav-interact);
8176
}
8277
}
8378

src/assets/sass/modules/_tables.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ tbody {
8282
}
8383

8484
&.selected {
85-
background-color: var(--clr-nav-hover-selected);
85+
background-color: var(--clr-nav-active);
8686
}
8787
}
8888

src/assets/sass/style.scss

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,7 @@
2525
--clr-a-alt: #9cd7ff;
2626
--clr-a-alt-interact: #e6f0ff;
2727
--clr-focus: hsla(216, 100%, 50%, 1);
28-
--clr-nav-hover: #9cd7ff;
29-
--clr-nav-hover-selected: #c1e4ff;
28+
--clr-nav-interact: #b1ddfd;
3029
--clr-nav-active: #e6f0ff;
3130
// neutral colours
3231
--clr-bkgd: #ffffff;
@@ -88,8 +87,7 @@
8887
--clr-a: #9cd7ff;
8988
--clr-a-interact: #bde4ff;
9089
--clr-focus: #ff00cc;
91-
--clr-nav-hover: #1b2f48;
92-
--clr-nav-hover-selected: #1b314f;
90+
--clr-nav-interact: #1b2f48;
9391
--clr-nav-active: #1c3456;
9492
// neutral colours
9593
--clr-bkgd: #373e45;

src/docs/design-patterns.html

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -138,11 +138,8 @@ <h3>Interaction</h3>
138138
<p><code>nav-active</code></p>
139139
<div class="design-patterns--color design-patterns--clr-nav-active"></div>
140140

141-
<p><code>nav-hover-selected</code></p>
142-
<div class="design-patterns--color design-patterns--clr-nav-hover-selected"></div>
143-
144-
<p><code>nav-hover</code></p>
145-
<div class="design-patterns--color design-patterns--clr-nav-hover"></div>
141+
<p><code>nav-interact</code></p>
142+
<div class="design-patterns--color design-patterns--clr-nav-interact"></div>
146143

147144
<p><code>focus</code></p>
148145
<div class="design-patterns--color design-patterns--clr-focus"></div>

0 commit comments

Comments
 (0)