Product preview card component

Community feedback

• 

  Nenad Cosovic•360

  @nenadmnePosted about 2 years ago

  Hello, i had some spare time so i look into your code and shere few thoughts.

  h1 { font-weight: bold; }

  // I would use just font-weight: 700; as you did in .category further bellow. //

  @media only screen and (max-width: 700px) { .prices { display: flex; align-items: center; }

  Here i saw you repeated some of the atributtes from previus code. When using @media you should write down only code that is changing further.

  Last thing i would suggest is that instead writting for example display:flex, for every class, you just tight them all under same line:

  .one, .two, .three, .four { display:flex; }

  nice project and happy coding =)

  1

  ---

  

  Muhamad Syah Reza•130

  @msyahrezaPosted about 2 years ago

  @nenadmne thanks for your corrections 😄

  1