Product preview card component

Community feedback

• 

  Douo•940

  @Douoo

  Posted over 1 year ago

  Hey Veronica, just came across your solution. Impressive - I like your solution and even better, I like your confidence. You did a very good job. A small suggestion though - in your <article> tag you gave it a fixed width of 21.4375rem, by doing this you are taking away the responsiveness and setting a constant width that stays the same for a certain condition (in your case responsiveness). Now there is no problem with this as long as it gets the job done but its not as flexible as it should be. Here is a better solution for it:

  article {
    width: min(21.4375rem, 85%); 
  ...
  }
  

  This will give it a max width of 21.4375rem but will keep the width at 85% in case its lower than the given width. By doing this you won't need to change or adjust the width property in different media size. Overall, you have done a pretty solid job for this challenge and seeing from your code I have no doubt that you will progress even better with time. Keep up the good work Veronica 👍🏽

  01
• 

  abiskar100•100

  @abiskar100

  Posted over 1 year ago

  @designver hey you did a wonderful job there but you missed the cart svg. <svg src="" /svg> . If you adjust the gaps on text-context it will be perfect .

  01