Skip to content
OverviewSolutions
10
Comments
2
Austin
@Ausranking

All comments

  • Austin•160
    @AusrankingPosted about 1 year ago

    Well done with a nice dev tool. Fully responsive, however, you may want to make the Ipad view grid-cols-2 for the cards, just to give it a better UI in the Ipad devices.

    1
  • Matheus Moreno•220
    @matheusmorenocfSubmitted about 2 years ago

    At first I put the product image in the html, but due to the responsiveness I realized that it would be easier to use the product as a background image so I could change it only with the css when the screen shrinks.

    I saw some tips in my last challenge, they said to use rem or em instead of px, is there any way to convert more easily?

    I stop using px in places like width and height too?

    I appreciate any tips that can help me.

    product-preview-card-component-main with css flex, sass

    2
    Austin•160
    @AusrankingPosted about 2 years ago

    Work well done @matheusmorenocf in your body style, I saw min-width:600px width:30% I suggest it better be max-width: 600px or width: 600px I suggest you include more semantics in your HTML such as "article" , "aside" and minimize the use of divs Good Job!

    0