@NehalSahu8055
Posted
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
- Remove the
commented line
below from thebody
as it makes no sense to us it.
/* width: 1440px; */
/* margin: auto;*/
-
Replace
height:100vh
withmin-height:100vh
to properly center the card. -
For
non-decorative images
give meaningful and descriptive alt likealt= "QR code to Frontend Mentor website"
. -
Use
Semantics
for the proper design of your code.
<body>
<header>
<nav>...</nav>
</header>
<main>...</main>
<footer>...</footer>
</body>
- Use
responsive units(rem, em, %)
from next project. Explore respective use cases on google.
link.
I hope you find this helpful.
Happy coding😄
Marked as helpful
@jasnoludek
Posted
Thank you for your feedback! I'm going to work on this now. @NehalSahu8055