@tesla-ambassador
Posted
Hey Tan Yi Hao, pretty neat solution! I am glad you had fun with this! In order to resolve your accessibility issues, you might want to consider the following:
- You might want to add a <main></main> landmark to contain your <div> tags this enables the browser to easily navigate the website. There are different types of landmarks your could use like <header> for the header section <main> for your main content < footer> for your footer All this is used to ease accessibility of your website by people who might be using screen readers and other features of the like.
- You might also consider adding an <h1> tag it's recommended to ease accessibility! In your case, you might add the <h1> on the "Improve your front-end skills by building projects" text. Happy coding!
Marked as helpful
@shashreesamuel
Posted
Just to add to the feedback regarding html semantic tags, here is a link to understand html semantics ,secondly the main reason why the html semantic tag main is recommended is because it describes the intended usage of the enclosed tags which will form the main content of the page.
Lastly all documents are required to have a h1 tag if headings are being used and it can only go up one-level meaning that if you have a h1 then the next tag in line should be a h2
I hope this helps strengthen the feedback given
Cheers
Marked as helpful
@tesla-ambassador
Posted
@TheCoderGuru Thanks for re-enforcing the feedback 💪
@tyihao
Posted
@tesla-ambassador @TheCoderGuru Huge thanks to the both of you for the taking the time to provide the invaluable feedback!
