
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Hassiai
Replace the section tag with a main tag to fix the accessibility issue.
A padding value for all the sides to .container and remove the padding value .img-card.
there is no need to give h1 and p separate divs. in the css give .card-content a padding value for all the sides and either give p a margin top value or h1 a margin bottom value. e.g.:
in the html <div class="card-content"> <h1></h1> <p></p> </div> in the css .container{ padding:1em; } .car-content{ padding: 1em; } .h1 { margin-bottom: 1em; }
Hope am helpful. HAPPY CODING
Marked as helpful
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