@MrLanter
Posted
Hi, this is a very good project and congratulations for this accomplishment!
I think I can give you some suggestions for improvements:
Text sizes should always be in rem
. Some people enlarge the text size in the browser settings and setting this size to px
might not respect their preferences. Here is a useful resource on this topic - 5 mins to read
It is true that it is not a huge project, but I think it's better to give less vague class names than container
or content
for the card.
Do you really need to wrap a <h1>
and a <p>
in a <div>
that is itself contained in a <div>
? I think it would be good to leave for example:
<div class="text">
<h1>Improve your front-end skills by building projects</h1>
<p>Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p>
</div>
without the .container
.
I hope this helps! Good luck for the future and have a nice day!