@Kaji17
Posted
Hi @Kaji17 👋, good job for completing this challenge and welcome to the Frontend Mentor Community! 🎉
Here are some suggestions to improve your code:
Try to use semantic tags in your code. More information here: With semantic tags:
<body>
<main class="main-content">
. . .
</main>
. . .
<body>
-
The profile images Alt Tags need to be improved. It should state the following; “image-qr-code“
-
I just wanted to encourage you to use
rem
as your main unit throughout the whole project instead ofpx
. There's nothing wrong with pixels, but they are not as responsive asrem
, especially if a user has a differentfont-size
in their browser other than the regular 16px. click here to access converter px to rem -
Use max-width: 15.625rem to .container selector instead of width.
-
Use padding: 1.25rem to .container selector.
I hope that help you 😃