Bryan Li• 3,550
@Zy8712
Posted
Your site looks great! The only changes I would make are:
- using a
<button>
instead of<a>
for the buttons, this is for accessibility reasons - if you want to use
<h1>
I suggest you instead of placing<h1>
for "Sedans", you should change it to<h2>
so all the columns have the same header. Then add an<h1>
header somewhere in your code that describes the page, you can then hide the header withdisplay: none
.
Aside from that your site looks great. Nice work 👍
Marked as helpful
0
scotttabor• 110
@scotttabor
Posted
@Zy8712 thanks for the tips! I was wondering how the headers should go in a situation like this, that makes a lot of sense.
0