Hey @Muaaz08 👋🏻
I have some suggestions to help you fix the accessibility issues and some other things.
- In your markup,
<div class="container">...</div>
should be<main class="container">...</main>
. This will fix the accessibility issues. Just, don't forget to generate a new repot once you fix the issues. - Next, instead of separately giving
border-radius
to each card, you can do the following:
.container {
border-radius: 10px;
overflow: hidden;
}
- Also, for the car icons, add
aria-hidden="true”
, because they are for decoration only. You can read more aboutaria-hidden
here. - I won't go into many details about resetting CSS, but I'll leave this cool article here, which will make more sense than my brief explanation.
- Lastly, I suggest adding
transition: all 0.2s;
to the button and the links, this will make:hover
smoother.
I hope this was helpful 👨🏻💻 you did a great job for the second project, nicely done. Cheers 👾
Marked as helpful
0