Shashi Lo• 1,345
@shashilo
Posted
Great job Ejiroghene. All the elements are there and it looks very much like the design. Your use of CSS Grid was on point. I did find some areas of improvement though.
- Pay closer attention to the details. Heading font size and content width.
- Overall max-width should be thinner.
- Cards are missing bottom
border-radius
,box-shadow
is too much, the top border should not be a border, instead it should be a pseudo element so it doesn't curve with the border-radius. - The entire project isn't horizontally centered.
1