Skip to content
Submitted over 2 years ago

Room Homepage

#accessibility
Karishma Garg•1,010
@karishma-dev
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Room homepage
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

Hey Guys, I just finished the Room Homepage Challenge. I would like you all to see it and let me know how I can improve it. Let me know my mistakes. Thank you!.

Community feedback

  • Shashree Samuel•9,260
    @shashreesamuelPosted over 2 years ago

    Good job Karishma completing this challenge

    Keep up the good work

    Your solution looks great however I have some feedback

    • The content to the right of the desk image needs to have some padding using padding

    • The text "Shop Now" needs to have some letter-spacing using letter-spacing

    In terms of accessibility issues, your document should have a level one heading thus I recommend changing this <h2 class="heading" id="headerHead"> Discover innovative <br />ways to decorate</h2> to <h1 class="heading" id="headerHead"> Discover innovative <br />ways to decorate</h1>.

    I hope this helps

    Cheers

    Marked as helpful
    0
    Karishma Garg•1,010
    @karishma-devPosted over 2 years ago

    Thank you for your feedback, I will fix these issues. :)

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord