Responsive Card Preview Page Using CSS FLEXBOX
Design comparison
Solution retrospective
Looking to imbibe the best practices as I progress, please let me know if there is a line of code you would have written differently. I take every suggestion into account.
Community feedback
- @Lukasz-MildePosted almost 2 years ago
Hi! Looks all good, well done. Few things to consider. DRY - Don't Repeat Yourself- try to think about how to write styles and use correct selectors to avoid repetitive lines as much as possible. You can do it by planning your CSS before touching the keyboard and in the future using SASS. @mixins are GREAT! Naming methodology such as BEM is a great addition. PX's, well px are ok in some cases like border-radius, but once you learn when and how to use REM/EM you will love it and it will be a game changer. Try to add more comments to your CSS file. The last thing is the Prettier code formater. I would say it's a must-have and yes again, you will love it. Other than this, the solution looks great, and again, well done!
0
Please log in to post a comment
Log in with GitHub
Join 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