Boris• 2,870
@mkboris
Posted
Nice work Ashwini202213, here are a few things to review
- All content should be wrapped within landmarks. Wrap a
main
tag around the .main-container and afooter
for the attribution. - You can use flexbox to properly center the card, add this on your
body
min-height: 100vh;
display: flex;
align-items: center;
flex-direction: column;
justify-content: center;
- Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset. This will help reset a list of default browser styles.
- Font-size should be written in
rem
not px
Marked as helpful
2
Ashwini Magar• 50
@Ashwini202213
Posted
Thank you! @mkboris
1