Loopstudios landing page

Community feedback

• 

  P

  Andrey•4,580

  @dar-ju

  Posted 4 months ago

  Hi Yacoub AlDweik!

  Great job!

  If you don't mind, I'll add a little bit from myself:

  • your close cross for the mobile menu didn't load, because of this there is no way to close menu
  • for the menu and social icons you use ul/li but for "Our creations" - div, I think you need ul there too
  • your keyboard accessibility works, that's cool! but for better readability there is should be aria-label for social icons
  • at the end of development it is necessary to do an overflow test for blocks and for text. For example, the "Our creations" block passes the test, everything is fine here. But the menu items and social network items - do not pass. Why is necessary to do test? Imagine that the customer wants to add new menu items, then they will start to go beyond the block boundaries. Here's how to fix it (the top menu for example): remove the margin-right from li, add the code for ul:

  flex-wrap: wrap;
  justify-content: flex-end;
  gap: calc(calc(var(--spacing)* 6)* calc(1 - var(--tw-space-x-reverse)));
  

  In this case, no matter how many menu items you add, the layout will always be stable.

  Everything else is great, good luck with your development!

  03

  Marked as helpful
• 

  Karol•3,100

  @VillageR88

  Posted 4 months ago

  You're ready for the job market!

  Just be sure to check the console for any errors.

  01

  Marked as helpful