Product Preview Card

Community feedback

• 

  P

  Erik S. Carlsten•290

  @ecarlste

  Posted 4 months ago

  You might consider using the <article> semantic html tag for components container rather than a <div>.

  The layout looks good and matches pretty closely to the example. I did notice a few of the fonts are a bit off. If you haven't already checked the Figman diagram for the CSS values for each element I'd suggest doing so. It makes it much easier to get the right values in there. Like for "Perfume", in tailwind v4 it should be text-xs to give you font-size: 12px rather than a font-size: 14px which is what text-sm give you in tailwind v4.

  I like your use of the re-usable <CodedBy /> component as you are likely using it multiple challenges! Nice job!

  10

  Marked as helpful