Design comparison
SolutionDesign
Community feedback
- @anoshaahmedPosted almost 3 years ago
To avoid accessibility issues in the future:
- wrap everything in your body in
<main>
OR giverole=""
to the direct children of your<body>
... Read more here - give ur
<img>
analt=""
<a>
should have anaria-label
... Read more here
Good job! :)
Marked as helpful0 - wrap everything in your body in
- @radriannPosted almost 3 years ago
Good job with this challenge! Some recomendations: The position of attribution, you can use the {position: absolute} for centrate it in bottom. Some alt attributes are empty, you shouldn't put them if you won't put anything in them. The hover icon can be reduced and it would look much better, plus a small transition of a few seconds can be added so that the change is not so abrupt.
Congratulations on completing the challenge ^ - ^ keep coding and have a nice day
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