HTML & SCSS and JS for mobile menu

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 3 years ago

  • the desktop design looks great but yes you still have to work on the responsive side of the site.
  • personally i prefer to use class instead of id's as they are reusable
  • the nav items i would suggest to have them in an unordered list to help with accessibilty

  Happy coding

  Marked as helpful

  0

  ---

  

  Андрій Рогов•1,155

  @Andrii-RohovPosted over 3 years ago

  @ChamuMutezva thanks))

  0
• 

  P

  Grzegorz Sterczewski•680

  @gsterczewskiPosted over 3 years ago

  Hello Andrii-Rohov 👋, my name is Grzegorz.

  Your solution looks super awesome on desktop, and it's alomost pixel-perfect with the design.

  One thing to check it's the responsiveness on screen width < 768px, because right now your layout does not fit on the screen.

  I'm sure it is just a small overlook.

  Overall, I am impressed with your work and animations that you add.

  Keep up the good work! 💪

  See you on the coding trail 😉

  Cheers!

  Marked as helpful

  0

  ---

  

  Андрій Рогов•1,155

  @Andrii-RohovPosted over 3 years ago

  @GSterczewski yeah.... in design files there was a mobile layout image, and it was 750px when usually it 350px, and i decided to do it only for width > 750...i guess i'm not done with this challenge yet)) thanks))

  0