The solution made with vanilla html and css
@NaveenGumaste
Posted
Hello Aliyah Regina ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
Increase the font-size of the "Title"
Warp your card content around the main tag Ex: 👇
<body>
<main class="container">
*all you content here*
</main>
</body>
h1 first and then h2, h3 and so onhappy Coding😀
Marked as helpful
@denielden
Posted
Hi Aliyah, I took some time to look at your solution and you did a great job!
Also add main tag and wrap the card for Accessibility and remove all unnecessary code, the less you write the better as well as being clearer: for example the <div class="qr-code"> container of qr image
Overall you did well :)
Hope this help and happy coding!
Marked as helpful
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
