Design comparison
Solution retrospective
are there any ways I can improve the css on this project?
Community feedback
- @correlucasPosted about 2 years ago
👾Hello @divakar54, Congratulations on completing this challenge!
Great code and great solution! I’ve few suggestions for you that you can consider adding to your code:
You did a really good work here putting everything together, something you can improve its your code html markup and semantics. You can replace the
<div>
that wraps each card with<article>
you can wrap the paragraph with the quote with the tag<blockquote>
this way you'll wrap each block of element with the best tag in this situation. Pay attention that<div>
is only a block element without meaning.This article from Freecodecamp explains the main HTML semantic TAGS: https://www.freecodecamp.org/news/semantic-html5-elements/
✌️ I hope this helps you and happy coding!
Marked as helpful0 - @milosshomyPosted about 2 years ago
Hey Divakar, congratulations on solution. I have few tips for you. You can use display: flex on your .rating-item div, that way the starts and the span will be next to each other like on design. You can translate your .rating-item:nth-child(2) , .rating-item:last-child so they move a little bit to the right like on design.
I hope this helps. Keep coding!
Marked as helpful0
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