Design comparison
Community feedback
- @HassiaiPosted almost 2 years ago
Replace <h3> with <p> and wrap <div class="attribution"> in a footer tag to fix the accessibility issues. for more on semantic html visit https://web.dev/learn/html/semantic-html/
Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units watch this https://youtu.be/N5wpD9Ov_To
give .start-card height: auto; to prevent the content from overflowing on smaller screens. give .content a with of 50%
For the overlay color on the image, give .image-container a background: color of soft-violet and display block and give .card-image a width of 100%, mix-blend-mode: multiply; opacity:0.7; there is no need to give .card-image a height value and . image-container position: relative and display flex and its properties.
Hope am helpful
Well done for completing this challenge. HAPPY CODING
0
Please log in to post a comment
Log in with GitHubJoin 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