Design comparison
SolutionDesign
Community feedback
- @oubaidelmoudhikPosted over 2 years ago
Good job on the challenge! I have a few improvement areas you could work on if you'd like: -Your main white background isn't showing, that's because your .container class got removed from the main element. -Also, to center everything on the page, try setting your body min-height to 100vh.
Anyhow, great work! keep it up and good luck on your journey!
0 - @romila2003Posted over 2 years ago
Hi Yewande,
Congratulations π on completing your second challenge, it was a great attempt. Some minor issues I want to address include:
- Within your
img
tag, you are missing an alternative text which is important for accessibility. If the image does not load, then users can still understand what the content of the image is. Also, the alternative text can help blind users using screen readers. The code will look something like this:
<img src=".images/image-qr-code.png" alt="QR Code Image" class="image">
- It is best practice to wrap your code with the correct semantics. For the container, it should be wrapped within the
main
tag and for the footer, it should be wrapped within thefooter
tag.
<main class="container"></main> <footer class="attribution"></footer>
Overall, great attempt and wish you the best of luck for your future projects so keep coding π.
0 - Within your
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