Refactor academy navigation bar

[RichDom2185]

Description

By moving the rendering logic to use conditional rendering instead of CSS classes, we can ensure correct margin behaviour (as the margin is dependent on the whether a child exists).

Ideally, we remove the old flexbox-grid dependency as well in the future.

Also refactored to use components instead of render functions.

Refer to commit history for more detailed list of changes.

Also resolves (partially) #2722.

Type of change

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] This change requires a documentation update
• [x] Code quality improvements

How to test

Checklist

• [x] I have tested this code
• [ ] I have updated the documentation

[coveralls]

Pull Request Test Coverage Report for Build 9062394319

Details

• 20 of 36 (55.56%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.02%) to 31.588%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/commons/navigationBar/NavigationBar.tsx	11	27	40.74%
<!--	Total:	20	36	55.56%	-->

Totals
Change from base Build 9062386750:	-0.02%
Covered Lines:	4943
Relevant Lines:	14716

---

💛 - Coveralls