@anoshaahmed
Posted
To avoid accessibility issues in the future:
<main> OR give role="" to the direct children of your <body> ... Read more here<img> an alt=""<a> should have an aria-label ... Read more hereGood job! :)
Marked as helpful
@radriann
Posted
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 helpful
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
