huddle landing page

Community feedback

• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted over 2 years ago

  Hey Ibrahim! This is a pretty solid solution, congratulations on completing the challenge. Here's a few pointers:

  • The website is pretty responsive but the components are too small on some screen widths therefore, you should consider changing the flex-direction to column at the screen width of 850px and maybe increse the width of the container containing you text and image to like 90% so that your content is easily visible.
  • You might want to add an aria-label = "text describing link" attribute to enable the browser to easily describe your link and this will fix your accessiblity issues.
  • You cannot contain the <main> landmark within a <header> because they are both landmarks that are meant to distinguish different sections of your code. This will fix your HTML validation issues.

  Happy coding and keep up the good work! 👍

  Marked as helpful

  0

  ---

  

  Ibrahim•180

  @Ibrahim-EltoukhyPosted over 2 years ago

  @tesla-ambassador thank you very much for your helpful comment

  0