Design comparison
SolutionDesign
Solution retrospective
Any feedback is welcomed! It is the best one i worked till now.
Community feedback
- @darryncodesPosted about 3 years ago
Hi Abhinay,
Nice work, really like your solution!
- I think your min/max heights are messing around with the responsive design on mobile in the verified buyer review
.user
cards. I'd stick to a max-width or even reduce the font-size. - your html and accessibility report looks a little scary but simple to fix. You could use aria-hidden="true" or declare a alt-text for your start image. Remove the
<h3>
just use a<p>
and change the<div>
in your.attribution
to a<footer>
tag
All the best!
Marked as helpful0@AbhinaygunjalPosted about 3 years ago@darryncodes ya i saw the accessibility report, it was all about minor mistakes, i will surely correct them and i will definitely try with the max-width thing. Even i thought of reducing the font size! Thanks for pointing it out! Thank you!
0 - I think your min/max heights are messing around with the responsive design on mobile in the verified buyer review
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