Lucas 👾• 104,440
@correlucas
Posted
👾Hello @Chidelemmanuel, congratulations for your first solution and 😎 welcome to the Frontend Mentor Coding Community!
Here's some tips to improve your project:
The card is a little bit big, the correct size in this case is around max-width: 320px
You've missed the page title, you can this to the <head> -- > <title>QR Code - Front End Mentor</title>
You've used too much divs to have the html structure, remember that you can build this with a simple approach, without using unnecessary divs, all you need is a single <main> or <div> to keep all the content inside, and nothing more. The ideal structure is the div and only the image, heading and paragraph.
See the structure below:
<body>
<main>
<img>
<h1></h1>
<p></p>
</main>
</body>
👋 I hope this helps you and happy coding!
0
