product-preview_card_component

Community feedback

• 

  cesar hernan ruscica•320

  @hernanruscicaPosted about 2 years ago

  Hello RickGatica, I really like your solution for this challenge. I think it meets the requirements, although it obviously doesn't look the same. The sizes of the fonts are others and the sizes of the containers. I really liked the fact that you used a single image. I did the same, although some people told me that I should have used the two provided, with the new "picture" tag that lets you change the image according to the "viewport". Another suggestion is that you use semantic tags instead of simple "div". It is also recommended to use relative units. And finally, I would put the fonts and everything related to the component, inside the component, not in the body, as you did with the fonts. Beyond that, the solution is great and I like it. Keep it up, you're on the right track.

  Marked as helpful

  0