Redesign Cleanup

[nilsstreedain]

Redesign Cleanup

Screenshots

Description

This PR primarily consists of cleanup and refinement of changes made by @Durbin-Elijah. Some additional changes have also been added, see below:

• Completed redesign and consolidated styling for Home and Courses pages
• Redesigned PageButton pop-up.
• Added favicon
• Refactored most of index.css into individual component style sheets
• Fixed broken logo bug
• Redesign navbar (top navbar and teacher sidebar)
• Fix student crash bug upon empty section (still need to address missing lectures)
• Fix broken student course links
• Update roster list view
• Update react bootstrap button styling to current color scheme (can't seem to change the default color of :active or :focus?)

Fixes #203

Checklist:

Before you submit your Pull Request, please make sure you have completed the following tasks:

• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.
• Any dependent changes have been merged and published in downstream modules.
• I have tagged my PR with the appropriate label(s).

[Durbin-Elijah]

Super clean! Everything looks super clean and navigates very well.
I only found 1 super tiny issue with the create course button overlapping when scrolling.

I think it should be a super simple fix and then it should be all good to merge.

[Durbin-Elijah]

I actually noticed 1 other thing I think the container for these elements need some margin at the bottom

[nilsstreedain]

Super clean! Everything looks super clean and navigates very well. I only found 1 super tiny issue with the create course button overlapping when scrolling.

I think it should be a super simple fix and then it should be all good to merge.

Ahh, for some reason it renders differently on chrome than safari, fixed it with z-index.

[Durbin-Elijah]

Super clean! Everything looks super clean and navigates very well. I only found 1 super tiny issue with the create course button overlapping when scrolling.
I think it should be a super simple fix and then it should be all good to merge.

Ahh, for some reason it renders differently on chrome than safari, fixed it with z-index.

Ahh it works a lot better now in that way, except I am now unable to click the button haha. Definitely seems like some weird interaction differences between the browsers with z-index, as I can see the landing header is covering it up when inspecting.

EDIT: It's not the header blocking it, since it's part of the header, its the courses div.

[nilsstreedain]

EDIT: It's not the header blocking it, since it's part of the header, its the courses div.

Oh man XD! Try now

[Durbin-Elijah]

Perfect, works great and look amazing, the website will be so visually appealing for the demo and expo with all these changes.