Responsive site made with Sass and pure JS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Grzegorz Sterczewski! 👋

  Good job! Your solution looks pretty good! 👏

  In addition to janez33's helpful feedback, I suggest,

  • Adding cursor: pointer to the mobile navigation menu icon.
  • Giving the slider buttons some more height in the mobile layout of the page to give the arrow icons inside them some space (as in the original design).
  • Preloading the slider images so that there is no delay between when the next slide is clicked and the image appears would be a nice touch!

  Keep coding (and happy coding, too)! 😁

  1
• 

  Janez Kolar•140

  @janez33Posted almost 4 years ago

  Hello I've looked at your solution :) I'm consider myself as a junior front-end developer and I have to say good job :) Here is one thing about responsivness - your solution doesn't look very good at 1000px viewporth width, there is too much white space at the bottom of the page. Setting media query to 1000px is not good for your solution I think and this is just my opinion. Your design doesn't look good on big screens from 1000px to 1600px. Width settings for .layout-wrapper doesn't do anything also height. Define your media queries before you start building the site (maybe look for some standard media queries you should always look) and then change positioning of the page to look good on those media queries and also device friendly. I hope I didn't say anything wrong. Keep building, every time you build something you are improving :) Regards from Slovenia

  1

  ---

  

  P

  Grzegorz Sterczewski•680

  @gsterczewskiPosted almost 4 years ago

  @janez33

  Thank you very much, your feedback is very helpful. I will address those issues soon. Have a great weekend. Hvala (I hope I get this right ) and cheers from Poland.

  1