@vanzasetia
Posted
👋Hi Haydn Kerr!
I have some feedback on this solution:
- For the try it free and get started, you should use anchor tag instead of button. In real website, it will navigate the user to a certain page.
- Try to use HTML semantic tags for wrapping each section, like using
section
. - Don't use
br
for spacing. You should do it in your CSS. - Use heading in sequence starting from
h1
,h2
,h3
and so on. The copyright should be a paragraph not ah6
. Don't useh6
to make it look small and bold 😉. - The about us, career and so on, should be a link, not just a list item.
That's it! Hopefully this is helpful!
Marked as helpful
@haydnkerr
Posted
@vanzasetia
thank you for the feedback, I wasn't aware that headings had a strict sequence to follow.
I've definitely got some updating to do