Hey @afewfirstnames 👋🏻
I have a quick tip to help you fix the accessibility issues and some other things.
- In your markup,
<div class="container row">...</div>
should be<main class="container row">...</main>
and this will fix the accessibility issue. Don't forget to generate a new repot once you fix it. - Next, the car icons, should have
aria-hidden="true”
, because they are for decoration only. You can read more aboutaria-hidden
here. - Also, perhaps you forgot to include, but the headings have different
font-family
. You can find the link instyle-guide.md
- I won't go into many details about resting CSS, but I'll leave this cool article here, which will make more sense than my brief explanation 🙃
- Lastly, I'd reduce
transition
from0.35s
to0.2s
, but it's just my personal opinion.
I hope this was helpful 👨🏻💻 Well done, you did a great job for the second project, keep it up. Cheers 👾
Marked as helpful
1