All comments

• 

  satish-maurya•20

  @satish-Maurya-4Submitted over 3 years ago

  I have written more code for service section, can same action could be done with less code.

  This is my very first project on this amazing platform.

  

  Responsive web app using css grid and flex box

  2

  30
  

  Darren Bawag•115

  @darrenbawagPosted over 3 years ago

  Tested the site. Nicely done. You nailed the styling :) good naming convention for classes/ids 🔥🔥

  Just don't forget the report for Accessibility issues and HTML issues.

  • provide alt tag for images <img class="arrow__icon" src="images/icon-arrow-down.svg" alt="arrow down" /> for screenreaders/accessability reason
  • I suggest to remove all # to all links with href and have it urls just like this

  <a href="https://facebook.com/"><i class="fa fa-facebook-square"></i></a>
  <a href="https://www.instagram.com/"><i class="fa fa-instagram"></i></a>
  <a href="https://twitter.com/"><i class="fa fa-twitter"></i></a>
  <a href="https://www.pinterest.com/"><i class="fa fa-pinterest"></i></a>
  

  That's it. Happy Coding! 😊

  Marked as helpful

  0

  This comment has received 1 replyShow