Interactive rating component with SASS
Design comparison
Community feedback
- @jakubjirousPosted almost 2 years ago
Hi Tales,
Overall, your code looks well-organized and readable. Here are a few suggestions for improvement:
1) Comment your code: It's always a good idea to add comments to your code, especially if you're working in a team. Comments help others understand your code and make it easier to maintain. Consider adding comments to explain the purpose of each section of your code.
2) Use descriptive names for classes and variables: Your class and variable names should describe what they represent. For example, instead of using the class name
"container"
, you could use"card-container"
to be more descriptive.3) Group related properties together: When you're defining styles for an element, group related properties together to make it easier to read and maintain. For example, you could group all of the styles for the
"ratings"
section together, rather than spreading them out.4) Use consistent formatting: Consistent formatting makes your code easier to read and understand. Make sure you're using the same formatting throughout your code, including indentation and spacing.
However, other than that, you did an excellent job!
Cheers, Jakub
Marked as helpful0@oliveiratalesPosted almost 2 years agoHi Jakub,
Thank you very much for the tips, I will apply it in my next projects!
Best regards, Tales.
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