Huddle landing page with a single introductory section built with Grid

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 11 months ago

  Hello there 👋. Good job on completing the challenge !

  Your solution looks excellent! I have a few small suggestions:

  • Avoid using overflow: hidden on the entire page as it may prevent scrolling on certain devices, affecting the user experience.

  • Consider using min-height: 100vh instead of just height. min-height allows the container to grow if necessary, preventing your solution from getting cut off on mobile devices.

  • For the logo image, it's recommended to use a more descriptive alt attribute, such as the company name, instead of simply "logo." This is helpful for accessibility.

  I hope you find it useful! 😄

  Happy coding!

  Marked as helpful

  1

  ---

  

  Mostafa Mohi•170

  @mo-oePosted 11 months ago

  @MelvinAguilar , Hi Melvin, I struggled trying to figure out why the page not scrolling when I put mobile mode, so it because of overflow, I didn't know also about the height thing updated my code now. Thank you very much for your help.

  0