Design comparison
Solution retrospective
Constructive feedback would be much appreciated.
Community feedback
- @brodiewebdtPosted almost 3 years ago
You need to fix the width of your card. It should be about 350px maximum, but yours grows well beyond that. I checked it all the way to 1900px. Set it to a max-width of 350px, and redesign it.
Wrap your container div in a Main tag and change the Equilibrium text to an H1. You have a class on it so you shouldn't have to restyle it. The text looks a little small now though. Add ALT text to all your images. You can leave it empty for these images as they are for decoration only. Doing these thing will clear the accessibility warnings. You are also using backslashes instead of forward slashes for the images.
Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/
Hope this helps.
0@Uyoyo-WPosted almost 3 years ago@brodiewebdt Thanks a lot. I'll make the corrections.
1
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