Skip to content
OverviewSolutions
15
Comments
2
Antoine Estievenart
@Kehs719

All comments

  • Antoine Estievenart•240
    @Kehs719Posted over 2 years ago

    This is a very nice submission @Odiesta, the size of the card is respected which is very nice.

    However there are a few things to improve to make it perfect :

    • For the image that changes depending the device's screen size you can create a div and add the CSS property background-image: url(./images/...) ;.

    • To avoid accessibility errors wrap your HTML body into a main tag :

    <body>
     <main>...</main>
</body>

    For the rest it is very good.

    Keep improving and happy Coding

    Marked as helpful
    1
  • Antoine Estievenart•240
    @Kehs719Posted over 2 years ago

    This is a very nice submission DevZardy, however,getting the accurate dimensions on the project would be great and would help achieve a better look alike to the images from FE mentors, you can do that by using the design images in Figma.

    For the image that change depending the device's screen size you can create a div and add the CSS property background-image: ;

    For the rest it is very good.

    Keep improving and happy Coding

    Marked as helpful
    0