Acknowledgements

• Good github workflow 💯
• No linters error 👍
• Clean and readable code

Changes Required

• Please make sure that when using the nav tag, you do so once. This is for semantics reasons.
  

  Portfolio-Project/index.html

  Lines 16 to 25 in 4cfe767

  	<nav class="mobile_nav">
  	<img src="./images/menu.png" class="icon" width="40" height="40" id="openMenu">
  	</nav>
  	<nav class="desktop_nav">
  	<ul class="desktop_nav_ul">
  	<li><a href="#work">Portfolio</a></li>
  	<li><a href="#about">About</a></li>
  	<li><a href="#contact">Contact</a></li>
  	</ul>
  	</nav>

• Kindly change your headings in your code. You used the h1 tag multiple times which supposed to be used for the first and most important heading. you can use either h2 or h3 to replace the ones below.
  

  Portfolio-Project/index.html

  Line 103 in 4cfe767

  <h1>Tonic</h1>

  
  

  Portfolio-Project/index.html

  Line 340 in 4cfe767

  <h1>Contact Me</h1>

Shaddai suggested that you use an anchor tag instead of the button so that when you hover on it it shows a pointer, and suggested that you use ul and li in place of divs and span in the card, and lastly to import a font and apply the font to the body tag.