Huddle landing page with single introductory section

Community feedback

• 

  faizan•2,420

  @afaiz-spacePosted over 2 years ago

  Hey @Kaushaljoshi29, congratulation on completing the challenges. I notice some issues.

  • replace height: 100vh; with min-height: 100vh; in body element
  • replace width: 60%; with width: 50%; in .conatiner-image class.
  • replace width: 35%; with width: 50%; and also remove margin-left: -2em from .conatainer-text class.
  • remove positon from footer element.

  Marked as helpful

  0

  ---

  

  Kaushal Joshi•100

  @Kaushaljoshi29Posted over 2 years ago

  @afaiz-space Thanks for your valuable suggestions, will implement them in the code.

  0
• 

  Ishpaul Singh•110

  @ishpaul777Posted over 2 years ago

  • The solution includes semantic HTML ✅
  • There are no accessibility issues ✅
  • The layout looks good on a range of screen sizes ✅
  • Code is well-structured, readable, and reusable ✅

  You have done incredible work implementing all the requirements and making the project similar to the design. All the best for your future projects. Keep up the Great work up!

  0

  ---

  

  Kaushal Joshi•100

  @Kaushaljoshi29Posted over 2 years ago

  @ishpaul777 Thanks for the appreciation

  0