@NehalSahu8055
Posted
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
- Remove
margins
from the.container
as this will not dynamically center the card. - To properly center the card.
- USING FLEXBOX
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
-
or alternatively
-
USING GRID
body{
min-height: 100vh;
display: grid;
place-items: center;
}
-
Use gradient
button:hover
-
background-image: linear-gradient(#6542FE,#342CE2);
-
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>
- Use
responsive units(rem, em, %)
from next project to increase responsiveness. Explore respective use cases on google.
link.
- Use
max-width
instead ofwidth
to make your design responsive.
Generate new screenshot after making the necessary changes.
I hope you find this helpful.
Happy coding😄
Marked as helpful
@Ajayfrizzy
Posted
Thanks so much although the justify-content is not working while align-items works. Also the min-height is not working fine, it taking the full height of the screen, instead of the normal content only. @NehalSahu8055
@NehalSahu8055
Posted
@Ajayfrizzy
You are using it in a wrong way, always use flex or grid on parent to center the child.
Marked as helpful