@danielmrz-dev
Posted
Hello @Chenxi96!
Your solution looks great!
I have a couple of suggestions for improvement:
- In order to make your HTML code more semantic, and since that is the main title of the screen, you can replace the
<h2>
with<h1>
. Unlike what most people think, it's not just about the size and weight of the text.
The <h1>
to <h6>
tags are used to define HTML headings. <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.
- Also, still about semantic HTML, you can replace your
div.container
withmain.container
. All these changes may have little or no visual impact but they make your HTML code more semantic and improve SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
Marked as helpful
@Chenxi96
Posted
Hi @danielmrz-dev,
Thank you for the feedback, I have updated this on my code!