Skip to content

Commit bc0c936

Browse files
committed
sidebar.scss: move sidebar styling out of layout.scss
Make sidebar.scss the single-source-of-truth and put all the styling related to the sidebar in sidebar.scss.
1 parent 5f368d5 commit bc0c936

File tree

2 files changed

+73
-72
lines changed

2 files changed

+73
-72
lines changed

assets/sass/layout.scss

Lines changed: 0 additions & 72 deletions
Original file line numberDiff line numberDiff line change
@@ -26,37 +26,6 @@ body {
2626
overflow: visible;
2727
}
2828

29-
aside {
30-
width: 218px;
31-
margin-bottom: 35px;
32-
position: sticky;
33-
top: 0;
34-
max-height: 100vh;
35-
overflow-y: auto;
36-
/* align with the top of the content */
37-
padding-top: 46px;
38-
}
39-
40-
#toc {
41-
border: 1px solid var(--main-border);
42-
padding: .5em;
43-
margin-top: 1rem;
44-
border-top: 1px solid var(--main-border);
45-
background: var(--sidebar-toc-bg-color);
46-
margin-right: 0.5rem;
47-
}
48-
49-
#toc ul {
50-
list-style-type: square;
51-
}
52-
53-
#toc li {
54-
/* use margin instead of line height for spacing because sometimes the
55-
headings wrap onto multiple lines */
56-
line-height: 1.2em;
57-
margin: 6px 0;
58-
}
59-
6029
#content {
6130
width: 702px;
6231
}
@@ -157,47 +126,6 @@ header {
157126
}
158127
}
159128

160-
// Navigation
161-
aside nav ul {
162-
margin-bottom: 1.4em;
163-
margin-left: 0;
164-
font-size: 16px;
165-
font-weight: bold;
166-
list-style: none;
167-
168-
li {
169-
margin-bottom: 0.5em;
170-
171-
a {
172-
color: var(--nav-link-color);
173-
174-
&.active, &:hover {
175-
color: var(--orange);
176-
}
177-
}
178-
179-
ul {
180-
display: none;
181-
margin-top: 0.3em;
182-
margin-left: 20px;
183-
font-size: 13px;
184-
font-weight: normal;
185-
186-
li {
187-
margin-bottom: 0;
188-
189-
a.active {
190-
font-weight: bold;
191-
}
192-
}
193-
194-
&.expanded {
195-
display: block;
196-
}
197-
}
198-
}
199-
}
200-
201129
.bottom-nav {
202130
@extend .callout;
203131
@include clearfix;

assets/sass/sidebar.scss

Lines changed: 73 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,37 @@ hr.sidebar {
99
@include background-image-2x($baseurl + "images/sidebar-divider", 218px, 12px);
1010
}
1111

12+
aside {
13+
width: 218px;
14+
margin-bottom: 35px;
15+
position: sticky;
16+
top: 0;
17+
max-height: 100vh;
18+
overflow-y: auto;
19+
/* align with the top of the content */
20+
padding-top: 46px;
21+
}
22+
23+
#toc {
24+
border: 1px solid var(--main-border);
25+
padding: .5em;
26+
margin-top: 1rem;
27+
border-top: 1px solid var(--main-border);
28+
background: var(--sidebar-toc-bg-color);
29+
margin-right: 0.5rem;
30+
}
31+
32+
#toc ul {
33+
list-style-type: square;
34+
}
35+
36+
#toc li {
37+
/* use margin instead of line height for spacing because sometimes the
38+
headings wrap onto multiple lines */
39+
line-height: 1.2em;
40+
margin: 6px 0;
41+
}
42+
1243
aside.sidebar {
1344
font-size: 13px !important;
1445
line-height: $base-line-height * 0.75;
@@ -29,6 +60,48 @@ aside.sidebar {
2960
}
3061
}
3162
}
63+
64+
// Navigation
65+
aside nav ul {
66+
margin-bottom: 1.4em;
67+
margin-left: 0;
68+
font-size: 16px;
69+
font-weight: bold;
70+
list-style: none;
71+
72+
li {
73+
margin-bottom: 0.5em;
74+
75+
a {
76+
color: var(--nav-link-color);
77+
78+
&.active, &:hover {
79+
color: var(--orange);
80+
}
81+
}
82+
83+
ul {
84+
display: none;
85+
margin-top: 0.3em;
86+
margin-left: 20px;
87+
font-size: 13px;
88+
font-weight: normal;
89+
90+
li {
91+
margin-bottom: 0;
92+
93+
a.active {
94+
font-weight: bold;
95+
}
96+
}
97+
98+
&.expanded {
99+
display: block;
100+
}
101+
}
102+
}
103+
}
104+
32105
// Floating sidebar button for tablet and smaller screens
33106
aside.sidebar.active {
34107
@include responsive-sidebar-ui;

0 commit comments

Comments
 (0)