Skip to content

Commit 702afab

Browse files
authored
enhance(curriculum): improve design (#11064)
* enhance(curriculum): add margin above "Resources:" * enhance(curriculum): use grey border on module hover * enhance(curriculum): update notes background-color in dark theme * enhance(curriculum): use same padding around notes * enhance(curriculum): adjust ul/ol/p spacing in notes * enhance(curricullum): reduce margin below list * chore(curriculum): update background color * chore(curriculum): use bg color from label bg * chore(curriculum): sync notes hue with bg-color * fixup! chore(curriculum): use bg color from label bg * fix(curriculum): revert --curiculum-bg-color in light theme
1 parent e6e2b5e commit 702afab

File tree

3 files changed

+27
-6
lines changed

3 files changed

+27
-6
lines changed

Diff for: client/src/curriculum/index.scss

+22-2
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,27 @@
228228
border: 0;
229229
border-radius: var(--elem-radius);
230230
margin: 1rem;
231-
padding: 0.5rem 2rem;
231+
padding: 1rem;
232+
233+
ol,
234+
ul {
235+
padding-left: 1rem;
236+
237+
li:last-child {
238+
margin-bottom: 0;
239+
}
240+
}
241+
242+
ol,
243+
ul,
244+
p {
245+
margin-top: 0.5rem;
246+
}
247+
248+
> p:first-child {
249+
// The "Notes:" paragraph.
250+
margin-top: 0;
251+
}
232252

233253
> :last-child {
234254
margin-bottom: 0;
@@ -251,7 +271,7 @@
251271

252272
ol,
253273
ul {
254-
margin: 1rem 0 2rem;
274+
margin: 1rem 0;
255275
padding-left: 2rem;
256276

257277
ol,

Diff for: client/src/curriculum/module.scss

+1
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353

5454
p.curriculum-resources {
5555
margin-bottom: 0.5rem;
56+
margin-top: 2rem;
5657

5758
+ ul {
5859
padding-left: 2rem;

Diff for: client/src/ui/base/_themes.scss

+4-4
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@
264264
--curriculum-color-topic-practices: #{$mdn-color-light-theme-violet-60};
265265
--curriculum-color-landing-laptop: #e3642a;
266266
--curriculum-border-color-list-item: #f2f1f1;
267-
--curriculum-border-color-list-item-hover: rgba(227, 100, 42, 0.7);
267+
--curriculum-border-color-list-item-hover: var(--text-inactive);
268268
--curriculum-bg-color-list-item-body: #fff;
269269
--curriculum-color-list-item-icon-topic-standards: #{$mdn-color-light-theme-red-60};
270270
--curriculum-color-list-item-icon-topic-styling: #{$mdn-color-light-theme-blue-60};
@@ -555,8 +555,8 @@
555555
--curriculum-color-topic-scripting: #f2cf8f;
556556
--curriculum-color-topic-tooling: #8ff2a4;
557557
--curriculum-color-topic-practices: #c891f2;
558-
--curriculum-bg-color: #664b33;
559-
--curriculum-bg-color-note: #321d13;
558+
--curriculum-bg-color: #321d13;
559+
--curriculum-bg-color-note: #332c29;
560560
--curriculum-bg-large-color: #343434;
561561
--curriculum-bg-color-topic-standards: #994b53;
562562
--curriculum-bg-color-topic-styling: #4e609e;
@@ -573,7 +573,7 @@
573573
--curriculum-color-landing-laptop: #b25c35;
574574
--curriculum-bg-color-list-item-body: #000;
575575
--curriculum-border-color-list-item: #4e4e4e;
576-
--curriculum-border-color-list-item-hover: rgba(227, 100, 42, 0.7);
576+
--curriculum-border-color-list-item-hover: var(--text-inactive);
577577
--curriculum-module-label-color: #e3642a;
578578
--curriculum-module-label-bg-color: #321d13;
579579
--curriculum-bg-color-list-item-topic-standards: #343434;

0 commit comments

Comments
 (0)