This looks pretty nice.
One thing to avoid though - you won't want to set body to be 100vw and 100vh. Viewport units don't take into account scrollbars, so is making your content off-center. Better to just give html and body a min-height: 100vh;
if you want to use viewport units.
I also suggest:
- not using the transform: translate method to center your content on the page.
margin: 0 auto;
would do. - making use of your grid at a smaller breakpoint. It stays single column for ages when there is plenty of space for cards to sit next to each other on mid size screens. You don't even need to use grid on mobile unless you really want to, or you could use a single column grid, then a 2 column at next breakpoint and then 4 column at largest breakpoint (I'm thinking of min-width mobile-first media queries here)
- Looking into grid-template areas. I think you'll like them :)
Happy coding!
@Basit-Dev
Posted
@grace-snow Thanks Grace....your feedback means a lot and I'm learning from it :-)