Design comparison
SolutionDesign
Solution retrospective
I would appreciate any feedback, thank you in advance.
Community feedback
- @Jos02378Posted about 3 years ago
Hey @VaNaChiMa, great job on this solution!
Some suggestions for you:
- You can use
text-transform: uppercase
to capitalized text in CSS instead of explicitly typing them in HTML. - For decorative images or icons, you don't need to put alt text for it since they don't actually add meaning to your content.
- You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
- You can try to use relative units like em in the future for padding. Here is a link to an article that explains when to use which unit read the article.
I hope this helps, good luck!
0 - You can use
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