Sunnyside agency landing page using HTML, CSS and JS

Community feedback

• 

  mjkjx34•20

  @MeHighLow34Posted about 3 years ago

  I think you use ::after or ::before for underline

  Marked as helpful

  1

  ---

  

  Mohsin Miyaji•440

  @mohsin316Posted about 3 years ago

  @MeHighLow34 good idea. ill try it as soon as a i can

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted about 3 years ago

  • use semantic elements where possible. <div class="ham"><img src="images/icon-hamburger.svg" alt=""></div> .This div will not be available to assistive technology users in its current state, it will require some more lines of code . The div and image are acting as a button to toggle the menu.
  • About, projects, contact, services , they should be anchor elements.
  • site is responsive, good job

  Marked as helpful

  1

  ---

  

  Mohsin Miyaji•440

  @mohsin316Posted about 3 years ago

  @ChamuMutezva thanks for giving your time and replying, what exactly should i use instead of the div with class of ham? i will look up About, projects, contact, services tags and their use case and replace the few links with these. thank youu

  0