Mobile first product card

Community feedback

• 

  Gustavo de Camargo Campos•200

  @themegazord

  Posted almost 3 years ago

  Hi Haydn Kerr, congratulations on the solution.

  I have some scores to make for you to improve more and more.

  1st Use classes and id in your HTML so that you can use them in CSS, this may not have been a problem, but imagine in a larger project, if you are guided only by HTML tags to assemble your CSS, you will become a monster kkkkk.

  2nd In your CSS, it might be easier if you use :root and declare the color variables for your project, so instead of writing the hsl every time you tell the browser that it will be called for example --primary -white or the color you want.

  Keep it up, you only tend to grow, congratulations.

  10

  Marked as helpful