Design comparison
Solution retrospective
Hey 👋 everyone Hope you guys are doing great! So from my last submission I have tried my best not to repeat those mistakes. This project took time and effort and would love if you could review and provide your valuable inputs for improvement. Thanks :)
Community feedback
- @eduardosgnPosted about 3 years ago
Hello! I'd suggest you try using SASS for your styling code, i'ts way cleaner and easier to write CSS using SASS mixins, nesting and variables!
1@dubeyaditya29Posted about 3 years agoThanks @eduardosgn I will learn it thanks for the info can you suggest some good resource to start with as a beginner.
0@dubeyaditya29Posted about 3 years agoHey @eduardosgn as you suggested I have learnt SASS from freeCodeCamp and will be using in the coming projects. Thanks for the suggestion. :)
1 - @YazdunPosted about 3 years ago
great job ! I suggest you to do mobile first, believe me it saves you a lot of headaches. another thing I noticed, display flex is a row by default so you don't need to declare it as a row unless you gave it column direction before.
I also opened a pull request to your repo which will fix some of these issues
1@dubeyaditya29Posted about 3 years agoThat's great @Yazdun thanks for this really didn't knew this before thanks :)
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord