Room Homepage

Community feedback

• 

  Luca Di Molfetta•365

  @dimolf345

  Posted about 4 years ago

  Hey! It's a wonderful project! I did the same using SASS and GRID. I think CSS Grid is the only way to make this project simple. I haven't seen your code, and since mine has surely more issues than yours, I honestly don't think I can judge it.

  I Just wanted to focus your attention on some things that I noticed:

  • The challenge requires the slider to responds also to user pressing left or right arrow to change images (Take a look at my project if you want);

  • When you change images, something strange happens: the previous one get loaded on the bottom of the page.

  Hope I was useful to you. Happy coding

  11
• 

  P

  Marko Nikolajević•1,095

  @MarkoNikolajevic

  Posted about 4 years ago

  Hi bimalmagar, you did a great job!

  I noticed that on mobile view the div with class='about__wrapper' goes under the image of the yellow chair.

  For the rest, everything looks good! Congrats and keep on coding!

  01