Hey @IvanaCliffords 👋🏻
I have some suggestions to help you fix the accessibility issue and some other things.
- First,
<div class="attribution">...</div>
should be<footer class="attribution">...</footer>
. This will fix the accessibility issue, but don't forget to generate a new repot once you fix the issues. - Next, add these to lines to fix the background:
.background-image {
background-repeat: no-repeat;
background-size: cover;
}
- For the card, and in general, avoid using fixed
height
s because they're not really responsive. - Instead of
height
you can do this:
.card-content {
padding: 4rem 4rem 6rem 2rem;
and this will make it more responsive and the card will look closer to the design.
I hope this was helpful 👨🏻💻 other than that, you did a great job for your second project, nicely done. Cheers 👾
Marked as helpful