@ringm
Posted
hey Kushank! Great job on completing this challenge, the site looks really good! Identical to the original! I have some minor suggestions that can make your site look even better:
-
The JS looks good, however I would encourage you to build some animation for the transition between slides. I would look much more polished and eye catching. [Here's my take on the challenge] (https://www.frontendmentor.io/solutions/sass-grid-and-css-animations-OG4YmHISD) if you want to check how I implemented them.
-
The site is not centered when you view it on large devices, I suggest adding the old
margin: 0 auto;
to the container class to achieve that ;)
Keep up the good work!
Cheers,
Martin.