Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Anna 1,060

    @annaindistress

    Posted

    I would advise you to add a border-radius to the container element. This will make it easier to find and change this value in one place if, for some reason in the future, it needs to be changed.

    .container {
        border-radius: 0.625rem;
        overflow: hidden; /* this hides image corners */
    }
    

    I also wanted to suggest taking another look at using the article tag for just the container element. According to its specifications, it's meant to represent content that can be taken from this page and placed somewhere else without losing its meaning. So, while the whole card could be an article, it might be better to use div for each individual part

    Marked as helpful

    0