Design comparison
Solution retrospective
Please give feedback. Thanks.
Community feedback
- @RayaneBengaouiPosted over 3 years ago
Hello szcompany,
Congrats for completing your first challenge ! 🙂
I would like to suggest :
-
Include
line-height
property to add spacing between lines in your text. -
Include
letter-spacing
property to your text to add space between letters
Add font-family: inherit to your buttons to override the default font for buttons.
-
Take a look at semantic HTML tags for better readability. Instead of using elements only, there are plenty of other tags that describe your code better.
-
Add border radius left top/bottom to your button.
Overall, well done for the challenge and happy coding ! 😃
1 -
- @szcompanyPosted over 3 years ago
Thanks for your feedback.
I will take a look at semantic HTML tags, during coding I have problem with that.
Thanks again.
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