CSS Grid for Card layout

[Xaviju]

I'd like to know why this repo is not using CSS grid for card layout instead of flex.

Current code displays the grid using flex as here: https://github.com/phuoc-ng/csslayout/blob/88200a698329be14ddc916053266ac788697de8f/client/patterns/card-layout/Details.tsx#L36

Instead, it would be simpler and more powerful to implement grid as:

.cards {
  display: grid;
  gap: 1rem;
  grid-template-columns: repeat(4, 1fr);
}

Am I missing something?

I'll be happy to PR this if you agree.

[Xaviju]

This issue affects also this pattern: https://csslayout.io/patterns/simple-grid/

[pragyamishra56]

@Xaviju If it would be possible then can you assign me to this issue I want to resolve this would be my first contribution

[Xaviju]

Sorry @pragyamishra56 this is not my repo. This issue was a suggestion to refactor an element layout that has never been responded by the repo owner. You should contact @phuocng

[pragyamishra56]

@Xaviju How to contact @phuocng to him
If you have any good first issues, could you please let me know Sir? I'm new to open source and eager to make contributions.