Responsive 3-column-preview-card-component using grid

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi Zaid! My name is Vanza!

  In my opinion, you're doing 👍 great job on using grid properties and the minmax() function. I have one tip for you. You can remove grid-template-columns: 1fr; and grid-template-rows: repeat(3, 1fr); since it is by default already one column and you can do multiple rows without specified it.

  .card-container {
      margin: 50px 5%;
      max-width: 100%;
      display: grid;
      grid-template-rows: repeat(3, 1fr);
      grid-template-columns: 1fr;
  }
  

  What I encouraged you to focus is that, keep practicing your CSS Grid Skill by doing challenge from Frontend Mentor until you comfortable using those grid properties.

  That's it! Hopefully this is helpful!

  Marked as helpful

  1

  ---

  

  Zaid•225

  @ZaidMarriePosted over 3 years ago

  @vanzasetia Hi Vanza, I am grateful for your feedback and your advice. Thanks for taking the time to take a look at my code.

  0