_nehal💎• 6,710
@NehalSahu8055
Posted
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
➨ To properly center the container.
-
Remove
margins
,paddings
andmin-height:500px
frombody
as this will not dynamically center the card. -
USING FLEXBOX
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
- USING GRID
body{
min-height: 100vh;
display: grid;
place-items: center;
}
➨ Use Semantics
for the proper design of your code.
<body>
<header>
<nav>...</nav>
</header>
<main>...</main>
<footer>...</footer>
wrap up .attribution div inside footer.
</body>
I hope you find this helpful.
Happy coding😄
Marked as helpful
0
Rozan-A• 240
@Rozan-A
Posted
@NehalSahu8055 thanks for your feedback same for you ^^
0