Responsive Landing Page using HTML, SCSS, Grid CSS, Flexbox and Js

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hi Okafor, good work on this challenge! 😉

  Here are a few tips for improve your code:

  • use semantic html like main, header, nav, button tags and wrap the content for improve the Accessibility
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the div container of image or container of text
  • add transition on the element with hover effect
  • the navbar in the mobile not work: the submenu not open and if i scrool down the page it's under the page contant, you can fix it adding z-index: 999 to .menu class

  Overall you did well 😁 Hope this help!

  Marked as helpful

  0

  ---

  

  P

  Okafor Daniel Onyebuchi•170

  @Buch-devPosted over 2 years ago

  @denielden Thanks for the feedback

  1