Skip to content

Commit e662c47

Browse files
authored
Merge pull request #15 from rakeshAlgo/nav-menu-ui
Fixing navigation style Issues
2 parents e839c60 + e89b756 commit e662c47

File tree

2 files changed

+41
-38
lines changed

2 files changed

+41
-38
lines changed

src/css/nav.css

Lines changed: 34 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,12 @@
4545
display: flex;
4646
word-break: break-all;
4747
align-items: center;
48-
font-size: 1.25rem;
49-
line-height: 28px;
48+
font-size: var(--font-small);
49+
line-height: 22px;
5050
/* font-family: "Open Sans", sans-serif; */
5151
font-family: "Source Sans Pro", sans-serif;
52-
color: var(--color-brand-gray1);
53-
font-weight: var(--weight-semibold);
52+
color: var(--color-brand-black);
53+
font-weight: var(--weight-normal);
5454
}
5555

5656
.nav-line .fas {
@@ -64,7 +64,7 @@
6464
}
6565

6666
.nav-menu a.nav-link {
67-
color: var(--color-brand-gray3);
67+
color: var(--color-brand-black);
6868
text-decoration: none;
6969
}
7070

@@ -97,7 +97,7 @@
9797

9898
.nav-menu .nav-list li a {
9999
font-size: 0.875rem;
100-
font-weight: var(--weight-normal);
100+
/* font-weight: var(--weight-normal); */
101101
font-family: "Source Sans Pro", sans-serif;
102102
}
103103

@@ -152,39 +152,42 @@
152152

153153
.nav-line[data-depth="0"] {
154154
padding: 0.625rem 1.25rem;
155-
color: var(--brand-color-gray2);
155+
color: var(--brand-color-gray1);
156156
}
157157

158-
/* .nav-item.is-active .nav-line[data-depth="0"] {
159-
background: #4287D60F;
160-
margin-bottom: 0.625rem;
161-
border-right: 4px solid rgba(66, 135, 214, 0.44);
162-
} */
158+
.nav-item .nav-line[data-depth="0"] > .nav-text {
159+
font-size: 1.25rem;
160+
line-height: 1.75rem;
161+
font-weight: var(--weight-semibold);
162+
color: var(--color-brand-gray1);
163+
}
163164

164-
/* .nav-item.is-active > .nav-line[data-depth="0"], */
165-
.nav-item.open > .nav-line[data-depth="0"],
165+
.nav-item.open > .nav-line,
166166
.nav-item.is-current-path.is-active-depth-2:not(.is-active) > .nav-line,
167167
.nav-item.is-current-page > .nav-line {
168168
background: #4287d60f;
169-
margin-bottom: 0.625rem;
169+
margin-bottom: 0;
170170
border-right: 4px solid rgba(66, 135, 214, 0.44);
171+
font-weight: var(--weight-semibold);
171172
}
172173

173174
.nav-item.is-active .nav-line[data-depth="0"] .nav-text {
174-
/* color: var(--color-brand-blue); */
175175
font-weight: var(--weight-semibold);
176176
}
177177

178178
.nav-item.is-active .nav-line[data-depth="0"] .fas {
179179
color: var(--color-brand-blue);
180180
}
181181

182-
.nav-menu .nav-item.is-current-page > .nav-line > a.nav-link {
182+
/* .nav-menu .nav-item.is-current-page > .nav-line > a.nav-link {
183183
color: var(--color-text);
184184
font-weight: var(--weight-semibold);
185-
}
185+
} */
186186

187-
.nav-item.open.is-active[data-depth="1"] .in-toggle[data-depth='1'],
187+
/* .nav-item.open.is-active[data-depth="1"] .in-toggle[data-depth='1'],
188+
.nav-item.open.is-active[data-depth="2"] .in-toggle[data-depth='2'],
189+
.nav-item.open.is-active[data-depth="3"] .in-toggle[data-depth='3'], */
190+
.nav-item.open.is-active > .nav-line .in-toggle,
188191
.nav-item.is-current-path.is-active[data-depth='1'] .in-toggle[data-depth='1'],
189192
.nav-item.is-current-page.is-active[data-depth='1'] .in-toggle[data-depth='1'],
190193
.nav-item.is-current-path.is-active[data-depth='2'] .in-toggle[data-depth='2'],
@@ -194,10 +197,15 @@
194197
transform: rotate(90deg);
195198
}
196199

197-
.nav-menu .nav-item.is-active[data-depth='2'] > .nav-line .nav-link {
198-
/* color: var(--color-brand-blue); */
200+
/* .nav-menu .nav-item.is-active[data-depth='2'] > .nav-line .nav-link {
201+
color: var(--color-brand-black);
199202
font-weight: var(--weight-semibold);
200-
}
203+
} */
204+
205+
/* .nav-menu .nav-item.is-active.open .nav-line .nav-text,
206+
.nav-menu .nav-item.is-active.open .nav-line .nav-link {
207+
font-weight: var(--weight-semibold);
208+
} */
201209

202210
.nav-menu .nav-toggle {
203211
background: none;
@@ -312,11 +320,7 @@ html.is-clipped--nav {
312320
}
313321

314322
.is-active[data-depth='0'] .is-active > .nav-line[data-depth="1"] {
315-
/* padding-left: 0; */
316-
/* justify-content: start; */
317-
/* flex-direction: unset; */
318323
font-size: 1.125rem;
319-
font-weight: 300;
320324
color: var(--color-brand-gray1);
321325
}
322326

@@ -328,14 +332,13 @@ html.is-clipped--nav {
328332
margin-left: 1.5rem;
329333
} */
330334

331-
.nav-menu .nav-item[data-depth='2'] {
332-
/* margin: 0px 0px 8px 0px; */
335+
/* .nav-menu .nav-item[data-depth='2'] {
333336
margin-bottom: 8px;
334-
}
337+
} */
335338

336-
.nav-menu .nav-item.is-active[data-depth='1'] > .nav-line .nav-link {
339+
/* .nav-menu .nav-item.is-active[data-depth='1'] > .nav-line .nav-link {
337340
color: var(--color-brand-blue);
338-
}
341+
} */
339342

340343
.nav-menu .nav-item.is-active.is-active-depth-2[data-depth='1'] > .nav-line .nav-link {
341344
color: var(--color-brand-gray1);

src/css/vars.css

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -113,13 +113,13 @@
113113
--line-height-base: 1.375rem;
114114

115115
/* Spacing variables */
116-
--base-space: 1rem;
117-
--base-small-space: 0.75rem;
118-
--base-medium-space: 1.5rem;
119-
--base-large-space: 2rem;
120-
--column-space: 2.5rem;
121-
--admonition-icon-space: 1rem;
122-
--admonition-content-space: 1rem;
116+
--base-space: 1rem; /* ~ 16px */
117+
--base-small-space: 0.75rem; /* ~ 12px */
118+
--base-medium-space: 1.5rem; /* ~ 24px */
119+
--base-large-space: 2rem; /* ~ 32px */
120+
--column-space: 2.5rem; /* ~ 40px */
121+
--admonition-icon-space: 1rem; /* ~ 16px */
122+
--admonition-content-space: 1rem; /* ~ 16px */
123123

124124
/* Heading fonts for responsive */
125125
--heading-h1-sm: 2.25rem; /* ~36px */

0 commit comments

Comments
 (0)