Skip to content
Submitted over 2 years ago

CSS and HTML

Gowsalaya•170
@Gowsalayadevi
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Stats preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi, I am not sure how to fit the image into the container in the desktop view. I would appreciate it if someone can review my code and give me the solution.

Community feedback

  • Arissa Hendricks•30
    @ajohnson29Posted over 2 years ago

    Great job overall! :)

    Here are a few things I would do for the image:

    .img-container { /* border-radius: 0 0.625em 0.625em 0; */ background: url(https://gowsalayadevi.github.io/stats-preview-card-1603/images/image-header-mobile.jpg) no-repeat; background-size: cover; border-radius: 0 10px 10px 0; }

    Remove the image from the HTML and add it as a background with background, background-size and border radius.

    Marked as helpful
    0
    Gowsalaya•170
    @GowsalayadeviPosted over 2 years ago

    @ajohnson29 That works perfectly!! Thanks, Arissa :)

    0
    Arissa Hendricks•30
    @ajohnson29Posted over 2 years ago

    @Gowsalayadevi You're welcome! ^_^

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord