@danielmrz-dev
Posted
Hello @PriyanshuPrama12!
Your project looks great!
I just have one suggestion about semantic HTML:
- I noticed that you used a
h4
for the main title.
The <h1>
to <h6>
tags are used to define HTML headings. Many people think it's just about the font-size, but it's not only that.
<h1>
defines the most important heading. <h6>
defines the least important heading.
Only use one <h1>
per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>
, then use <h2>
, and so on.
I hope it helps!
Other than that, great job!
Marked as helpful
@Priyanshu-WD
Posted
Thank you @danielmrz-dev, this is really helpful. I am going to improve my code.