Design comparison
Solution retrospective
Edited to match figma design 1:1 as closes as possible. All feedbacks are welcome :D
Community feedback
- @denieldenPosted about 2 years ago
Hello asdiAdi, You have done a good work! π
Some little tips to improve your code:
- use
main
tag to wrap the card and improve the Accessibility but not as a container of that one element - also you can use
article
tag instead of a simplediv
to the container card for improve the Accessibility - use
min-height: 100vh
to body instead ofheight
, otherwise the content is cut off when the browser height is less than the content - 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!
Marked as helpful1 - use
- @adebisiabdussamadPosted about 2 years ago
Cheers asdiAdi π₯ 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="./assets/icon-ethereum.svg" >
-This is how it should be:
<img src="./assets/icon-ethereum.svg" alt="">
the second one with the alt attritube. to know more about the alt attribute you can click this link Img elementYou need to apply the alt for both of your images 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