Room homepage | FLEXBOX & JS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Dwight! 👋

  Good work on this challenge! 👍

  The desktop layout looks good and the slider works well! (And I noticed you made the navigation links link to your Instagram page... 😆.)

  I suggest,

  • Doing a little more work on the responsiveness of the site. Currently, the text content of the page takes up the entire width of the screen when the width of the viewport decreases, creating a lot of empty white space to the right side of the screen and making the text slightly less readable. Perhaps you could keep the text content as it is in the desktop layout and allow the width of the images to decrease instead (while making sure that they are cropped and do not distort when the width of the page gets smaller). Then at some point, you could break into a single-column layout where both the images and the text take up the entire width of the viewport.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Dwight•215

  @SarsiPCPosted almost 4 years ago

  @ApplePieGiraffe I have improved the responsiveness of the website! You should check it out, once more.

  Thanks for your suggestion, man!

  0

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  @SarsiPC

  Looking better now, Dwight! 👍

  BTW, I like the way the mobile navigation slides into the page! 😉

  Keep coding! 🙌

  1