Responsive site made with Sass and pure JS

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)! 😁

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