Design comparison
SolutionDesign
Community feedback
- @polukarpPosted about 2 years ago
Hey, Ken. Great solution overall, but I could suggest several things to upgrade:
- Add fonts that are listed in style-guide.md (it would look even better if you apply them)
- Use
aspect-ratio: 1/1;
for your .qr-image container so that it is the same ratio and size as your img element. That would be important if you would want to make a scale hover effect or something like that, because your image would overflow a liitle on the bottom, so consider using aspect-ratio property for that. Here's a link if you want to learn more about it.
Marked as helpful1
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