@VCarames
Posted
Hey there!👋 Here are some suggestions to help improve your code:
-
The background color is incorrect (unless if you did this on purpose).
-
To center you content to your page, add the following to your Body Element:
body {
min-height: 100vh;
display: grid;
place-content: center;
}
-
The car images/icons in this component are purely decorative; They add no value. So their Alt Tag should left blank and have an aria-hidden=“true” to hides them from assistive technology.
-
Your "buttons" were created with the incorrect element. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.
-
The button styling is incorrect; it should be a solid white.
-
Your buttons need to have
cursor: pointer
,:hover
and:focus-visible
.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful
@akshaykumarmondal
Posted
@vcarames Thanks comrade!! cursor: pointer, :hover and :focus-visible ---> I did not get this one.