Design comparison
Solution retrospective
Another completed challenge!
All feedback is welcome!
Community feedback
- @joehaddad1000Posted 9 months ago
Here's my feedback on your solution:
-
Semantic HTML: Your HTML is well-structured and makes good use of semantic elements. The
main
tag is used to wrap the main content, and thediv
tags are used appropriately for grouping elements. Theimg
tag is used for the image, and thea
tag is used for the links. Good job! -
Accessibility: The
img
tag is missing analt
attribute, which is important for accessibility. Screen readers use thealt
attribute to describe the image to users with visual impairments. Consider adding a descriptivealt
attribute to theimg
tag. -
Responsive Design: Without seeing the CSS for different screen sizes, it's hard to say how well the layout would adapt to different devices. Consider using media queries to ensure your layout looks good on a range of screen sizes.
-
Code Structure: Your code is well-structured and readable. The CSS is organized with clear, descriptive class names, which makes it easy to understand what each style rule is doing. The use of CSS variables for colors is a good practice as it improves maintainability.
-
Improvements: Consider adding
:focus
styles to your links for better keyboard accessibility. Users who navigate with a keyboard or other input device should be able to see which element has keyboard focus.
Overall, your solution is well-implemented with a few areas for improvement. Keep up the good work!
Marked as helpful1@ValperDevPosted 9 months ago@joehaddad1000 I will keep that in mind for future projects thanks!
1 -
- @danielmrz-devPosted 9 months ago
Hello there! šš½āāļø
Your solution looks excellent!
I have a suggestion about your code that might interest you:
š If you don't have the Figma design files, I recommend using a browser extension called Perfect Pixel.
It allows you to compare your finished project with the design images that come along when you download the project and check the (almost exact) dimensions. It's very useful!
I hope it helps you in future projects! š
Other than that, great job!
Marked as helpful1
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