@Victor-Nyagudi
Posted
Good job on this one.
I didn't see any issues in the HTML or CSS. I also like that you used the BEM naming convention for your classes, which is good for organization.
Good job on using custom properties and naming them well. A lot of people don't pay enough attention to using proper names for classes and CSS properties which can lead to small, unnecessary problems down the line.
I noticed the card isn't centered on the screen.
You could try making the <body> a flex container i.e. display: flex then align and justify it to the center. You can then add flex-direction: column since there are two items inside.
body {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}
Try it, and let me know how it goes.
All the best with future solutions.
@narahkaruna
Posted
@Victor-Nyagudi
I'll look into it.
Thank you.
