Design comparison
SolutionDesign
Solution retrospective
Hi there,
here's my next challenge on Frontendmentor.
Taking my first steps into JavaScript with this one, and working on improving my CSS and HTML skills.
Grateful for any comments and tips that will help enhance and refine this solution.
Thanks in advance!
Community feedback
- @Zy8712Posted 10 months ago
Your site looks pretty good. Only things I'd alter/change are:
- the use of
<ul>
and<li>
for the buttons is sort of redundant in my opinion. You could just contain the five buttons in a<p>
tag and usedisplay: flex
justify-content: space-between
(as you did with the<ul>
) - centering the text of the rating buttons and applying font weight to it
Aside from that your site is very solid. Great work!!!
Marked as helpful0@SebBudynskiPosted 10 months agoHi @Zy8712 ,
I wasn't sure whether to use <li> on the buttons.
I made the changes right away.
Thank you for your comment.
0 - the use of
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