asdiAdi• 130
@asdiAdi
Posted
I think it's better to put some alt text on the image. Also make the card fixed width so it won't shrink on smaller screen sizes.
1
Mohit Sahoo• 20
@A-Realist
Posted
@asdiAdi Understood. Thank you for advice.
0
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
After a long break, I'm finally starting to learn Web Dev and I'm quite happy with the result. I'll try to make other projects mobile-responsive.
What specific areas of your project would you like help with?
There are mainly 2 problems that I was unable to solve:
-
The QR code image has some space from the top which I'm unable to get rid of. I've a padding of 25px which gave a nice margin to the entire card but the top has a little more space than required.
-
The text inside tag: is not aligned like it is in the original design. I could have used 3 tags but maybe it wasn't the best practice.
Join 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