1`st product preview card component

Community feedback

• 

  Franco Juarez•220

  @JuarrisonPosted about 2 years ago

  Hi @MariusSB-code. Congratulations for your work. First of all, I think you could explore some semantic html tags. Here is some relevant information about it: https://www.w3schools.com/html/html5_semantic_elements.asp

  Secondly, I think that instead of creating a special container for the image and applying the background-image property, it is more convenient to work with the image as an element itself and discard the container. On the other hand, you could use the flex-direction: column property for the mobile version and, with the application of media queries, change this property to flex-direction: row for the tablet and desktop versions. Greetings, I hope this comment has been useful.

  Marked as helpful

  0