@NehalSahu8055
Posted
Hello Coder 👋
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
- Use
min-height:100vh
instead ofheight:100vh
Todynamically center the card
body{
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
...
}
- Use Semantics for the proper design of your code.
<body>
<main>... main content goes here ...</main>
<footer>... .attribution div goes here ... </footer>
</body>
-
For
non-decorative images
give meaningful and descriptive alt likealt= "QR code to frontend mentor website"
. -
Use
responsive units(rem, em, %)
from next project. Explore respective use cases on google.
I hope you find this helpful.
Happy coding😄
Marked as helpful
@SarahCooper-TC
Posted
@NehalSahu8055 Thank you for your feedback! This helps! I will explore responsive units (rem, em, %) on google!