Design comparison
SolutionDesign
Solution retrospective
Any suggestions?
Community feedback
- @MelvinAguilarPosted about 2 years ago
Hi @Vinzz34 ๐, good job completing this new challenge! ๐
I have some suggestions you might consider to improve your code:
- The alternative text must not contain hyphens, underscores, or the words "image" or "photo", it must be human readable.
- To make alternative texts more worthwhile, add descriptive text to the alt attribute of the QR image to explain what the QR image does. Upon scanning the QR code, you will be redirected to the frontendmentor.io website, so an example of alternative text would be "QR code to frontendmentor.io". You can read more about alternative text here.
- Use
width: 100%;
instead ofmax-width: 29rem
in the image selector to make responsive images.
- Use
margin: 0.938rem
in the.container
selector to add some space for the container card and the screen edge on mobile devices.
Above all, the project is done well๐.
Good job, and happy coding! ๐
Marked as helpful3
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