Design comparison
SolutionDesign
Community feedback
- @mkborisPosted 3 months ago
Hi @Tcode2024, great job completing your first challenge here are a few things to review
- Use a separate file for your CSS as it improves maintainability.
- All content should be wrapped within landmarks. Wrap a
main
tag around the .container and afooter
for the attribution. - The text content in these
div
elements should be semantically wrapped in more appropriate HTML tags like headingsh2
or paragraphsp
for better accessibility and structure. - Font-size should be written in
rem
not px
Marked as helpful0@Tcode2024Posted 2 months ago@mkboris Thank you for your constructive feedback, I shall update my code to incorporate those key points.
1 - @brunohenriquedeveloperPosted 3 months ago
Congratss, you did really well!
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