huddle-landing-page-with-curved-sections-master

Community feedback

• 

  Mennatallah Hesham•1,490

  @Mennatallah-HishamPosted about 1 year ago

  Hi Dipesh,

  you did great 👍, i ;like that you used aria-label for social links 👏

  Here are some suggestions to improve your code:

  Alt

  • for the logo img , it is better to add the website name/ brand name

  <img src="./images/logo.svg" class="header-logo" alt="huddle">
  

  • for decorative img / illustrations you can use aria-hidden="true", which will skip the image for screen reader users.

  contact info

  • you can use Email link, which is a <a> which gives you the option to specify an email address to send an email.

  <a href = "mailto: [email protected]">[email protected]</a>
  

  • you can also link to a telephone number using <a> by using tel:

  <a href="tel:+4733378901">
  

  <article>

  • most of your divs can be replaced with <Article> for better semantics, you can read more about <Article> here

    Hope you find this helpful, Happy Coding

  0