Design comparison
Solution retrospective
Hi everyone ! Any feedback would be appreciated Thank you !
Community feedback
- @denieldenPosted almost 2 years ago
Hello Soles, You have done a good work! π
Some little tips to improve your code:
- add
main
tag and wrap the card for improve the Accessibility - add
margin: 0
to the body to remove the scroolbar of browser - also you can use
article
tag instead of a simplediv
to the container card for improve the Accessibility img
element must have analt
attribute, it's very important!- You can fix the effect
:hover
creating adiv
that appears on hover. I used tailwind but you can still see and understand which css properties you can use to do the same. Look here -> my solution - Using
<hr>
for the line is not the best way because this tag have a semantic meaning... in this case use div withborder-bottom
because this line is decorative - add
transition
on the element with hover effect - instead of using
px
use relative units of measurement likerem
-> read here
Keep learning how to code with your amazing solutions to challenges.
Hope this help π and Happy coding!
0 - add
- @adebisiabdussamadPosted almost 2 years ago
Cheers soles π₯ on completing this challenge.. .
An img element must have an alt attribute. This enables physical challenge to understand the image better.
-This is your code:
<img src="images/icon-view.svg" id="view">
-This is how it should be:
<img src="images/icon-view.svg" id="view" alt="" >
the second one with the alt attritube. to know more about the alt attribute you can click this link Img element your code is really amazing...hope you find this helpful...
Happy coding!!!ππ
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