Manage Landing Page solution

Community feedback

• 

  leodev•570

  @hangtime319

  Posted 19 days ago

  Congratulations!!

  10

  Marked as helpful
• 

  P

  Andrey•4,580

  @dar-ju

  Posted 21 days ago

  Hi Mohamed Biomy!

  Great job, but I see you don't have a tablet version, with a screen width of 600-1100px the content doesn't fit on the screen. You don't necessarily need a design layout for this, it's enough to just move and compress the blocks.

  What can be added\improved:

  • for better semantics for all content that is between the header and footer, you need to use the <main> tag, for the menu there is a <nav> tag
  • where the listing 01, 02, 03 is a list of ol\li
  • the logo in the footer is a different color
  • the list of social networks is also a ul\li list
  • the location of the blocks inside the footer differs from the layout
  • if clicking on something causes some action, then it is better to use <button> for this, you need to fix it for the burger and close burger buttons
  • the avatar images do not have alt, this is a error, alt can be left empty, but it should be

  Layout is precisely attention to detail.

  Good luck with your future projects!

  10

  Marked as helpful