Design comparison
Community feedback
- @lkosterPosted 5 months ago
Great job on your solution! Here are a few points I noticed:
Semantic HTML: You used appropriate tags like <main>, <section>, and <ul>, which helps with the structure and readability of your code. However, consider using <h2> instead of <h4> to maintain a proper heading hierarchy.
Accessibility: The alt text for the avatar image is a good start. You might want to ensure that the color contrast for your links when hovered meets accessibility standards. Adding aria-labels to links can also improve screen reader accessibility.
Responsiveness: Your layout looks good, but it could benefit from some media queries to handle different screen sizes better. Using relative units like % or rem for widths and heights can also improve flexibility.
Code Structure: Your code is well-organized and easy to read. Using CSS variables for colors can make future theme adjustments easier and keep your CSS DRY (Don't Repeat Yourself).
Overall, your solution is solid and functional. With a few tweaks, it can be even more accessible and responsive.
Keep up the great work!
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