Skip to content
Submitted about 4 years ago

Room homepage using Vanilla JS, CSS/Flexbox

Awais Muhammad Ahmed•120
@awaiskorai
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

Worked really hard on this, first ever Vanilla Javascript based page. Tried my best with the slideshows and all. I would really love a feedback. And sorry for the messy pictures for desktop version, I thought it was not worth the effort after I woke up all night lol. Would love a feedback for my css, javascript code and coding practices.

Happy Coding :)

Community feedback

  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted about 4 years ago

    Hey, there, Awais Muhammad Ahmed! 👋

    Good job on this challenge! Your slider works well and your page is responsive! 👏

    I suggest,

    • Adding a hover state to the links in the header of the page.
    • Setting the object-fit property of the images on the page to cover (or doing something else similar) so that the images maintain their aspect ratio when the page is resized and do not look stretched.

    Keep coding (and happy coding, too)! 😁

    1
    Awais Muhammad Ahmed•120
    @awaiskoraiPosted about 4 years ago

    You are certainly my favorite coding giraffe here. Always love your feedbacks ❤️ Would certainly do the above things. Thank you.

    Happy Coding, el mentor 😉

    1
    ApplePieGiraffe•30,545
    @ApplePieGiraffePosted about 4 years ago

    @awaiskorai

    Glad to help! 👍

    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