Design comparison
Community feedback
- @sakthivel155Posted 3 months ago
Hello @J-Jawad,
Great job on the code! A couple of suggestions:
-
Avoid using global styles like
display: flex; align-items: center; justify-content: center; flex-direction: column;
on thebody
tag—it can cause issues later on. Consider applying these styles to a more specific container. -
Class names should reflect their purpose, not content. Instead of using names like
karma-card or team-builder
, opt for something more descriptive of their function, likecard-container or layout-flex
. Also, if you don't style these classes in CSS, it's better not to include them.
Hope this helps!
Thank you.
0@J-JawadPosted 3 months ago@sakthivel155 Thanks a lot!! Definitely will look out for those details next time.
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