Hey @uzzii-21 👋🏻
I have some feedback for the project.
- For the car icons, add
aria-hidden="true”
, because they are for decoration. You can read more aboutaria-hidden
here. Like this:
<img src="./dist/images/icon-sedans.svg" alt="icon-sedans" aria-hidden="true”>
if you noticed, I removed these width="100" height="100"
inline styles, try to avoid inline styling because it's considered bad practice.
- Also, I suggest adding
transition: all 0.2s;
to the button and the links, this will make:hover
smoother and addcursor: pointer
as well. - Lastly, instead of using
height
try usingmin-height
it is more responsive since it allows the body to grow if needed.
I hope this was helpful 👨🏻💻 other than that, you did a great job, well done. Cheers 👾
@uzzii-21
Posted
Hello @kens-visuals 🤝
I appreciate you for pointing out my mistake. And I will definitely solve those problems.
Thanks for reviewing my project and giving me a great feedback 🤩
@uzzii-21 you're welcome 😇 I'd really appreciate if you could mark the comment as helpful 🙃
Marked as helpful