@vanzasetia
Posted
Hello, Eren! 👋
Replace all the <h1>
with <h2>
. There should not be more than one h1
on a page. Many <h1>
elements mean many titles which can confuse the users, especially the screen reader users.
You should use CSS to uppercase the text instead of doing it manually—<h1>SEDANS</h1>
. Screen readers might spell the uppercased word in the HTML (spelled letter by letter).
Use interactive elements for the "Learn more" buttons instead of using <div class="btn">Learn More</div>
. Interactive elements can either be links or buttons.
For your information, anchor tags are for navigation. The <button>
element is for an action like opening a modal, submitting a form, toggling an element, etc. It is essential to use the correct elements.
All the car icons are decorative images. I recommend leaving the alternative empty.
For your information, decorative images are images that don't add any information and serve only aesthetic purposes.
You should not wrap each element inside the card with <div>
elements especially when they don't have any styling. I recommend removing them.
Add rel="noopener"
to all links with target="_blank"
. It helps protect users of legacy browsers from security issues. Read more — Links to cross-origin destinations are unsafe
I hope you find this useful. Have fun coding! 😄
Marked as helpful