Design comparison
SolutionDesign
Solution retrospective
The solution made with vanilla html and css
Community feedback
- @NaveenGumastePosted over 2 years ago
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>
- Always use
h1
first and thenh2
,h3
and so on
happy Codingπ
Marked as helpful1 -
- @denieldenPosted over 2 years ago
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 imageOverall you did well :)
Hope this help and happy coding!
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