Design comparison
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Kixzy, congratulations for your new solution!
๐จโ๐ป Hereโs some tips to improve your solution code:
Remove all the margins and
width: 1440px
(this is only a reference to say in which dimension the design were created. Addmin-height: 100vh
andflexbox
to align it, here's your code fixed:body { min-height: 100vh; display: grid; /* width: 1440px; */ /* margin-top: 100px; */ /* margin-left: 100px; */ justify-content: center; background: var(--bg-color); justify-items: center; align-items: center; }
Alt text
<img src="./images/image-qr-code.png" alt="#">
Don't forget to use the
alt
text to allow screen readers to recognize that img. Adding alternative text to photos is first and foremost a principle of web accessibility. Visually impaired users using screen readers will be read an alt attribute to better understand an on-page image.โ๏ธ I hope this helps you and happy coding!
0
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