Skip to content

Commit

Permalink
Update Navigation Bar (#307)
Browse files Browse the repository at this point in the history
* Update Navigation Bar

Updated the Navbar to follow the order
given by PageOrder in globalTypes.ts

* Update Sidebar.scss

Changed the dropdown-header to flex instead of block
allowed for all headings to be centered vertically

* Update Sidebar.scss

Fixed the lint error, i didn't alphabetically
order the css
  • Loading branch information
lawtlee authored Oct 13, 2023
1 parent a37b5b0 commit d35133c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
9 changes: 5 additions & 4 deletions src/components/Sidebar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -148,15 +148,16 @@ export default function Sidebar(props: sidebarProps): JSX.Element {
<div className="dropdown-header">
<a href={PageURLs.POINTER_MOTIVATION}>Why Pointers?</a>
</div>
<Dropdown title="Lessons" list={lessonList} />
<Dropdown title="Exercises" list={exerciseList} />
{/* TODO: delete Demo and Error links before production! */}
<div className="dropdown-header">
<a href={PageURLs.DEMO}>Demo</a>
</div>
<div className="dropdown-header">
<Dropdown title="Lessons" list={lessonList} />
<Dropdown title="Exercises" list={exerciseList} />
{/* TODO: delete Demo and Error links before production! */}
{/* <div className="dropdown-header">
<a href="/error">Error</a>
</div>
</div> */}
</div>
);
}
3 changes: 2 additions & 1 deletion src/styles/Sidebar.scss
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,10 @@ $sidebar-width: 357px;
}

.dropdown-header {
align-items: center;
color: $text-color;
cursor: pointer;
display: block;
display: flex;
font-family: $play;
font-size: 24px;
height: 47px;
Expand Down

0 comments on commit d35133c

Please sign in to comment.