Congradulations on finishing this project. Here is what i think on areas that might need some attention.
- The h1 tag is supposed to be the first heading in a page. Starting with h2 and then going to h1 is considered to be bad practise . Avoid using heading tags to resize text - you can accomplish that using CSS font-size. For reference see "https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements " 2.I propose to use anchor tags in your navigation(nav-link class) as that will be the norm in a website.
1