mk-374• 40
@mk-374
Submitted
Yessica-jsy
@Yessica-jsy
All comments
- Yessica-jsy• 40
@Yessica-jsy
Posted
Semantic HTML: Consider using more semantic HTML tags, like <article> or <section>, to wrap the main content. This improves accessibility and search engine optimization.
Heading Levels: The <h4> tag is used for the title, but it's important to consider the overall heading structure. Make sure it logically follows the hierarchy, especially if <h1>, <h2>, or <h3> tags are used elsewhere.
1 @Wrajt
Submitted
Yessica-jsy• 40@Yessica-jsy
Posted
The code generally works well, but it seems a bit disorganized. I noticed that the <footer> is outside of the <body>, which is not correct. The <footer> should be inside the <body> to maintain a valid and semantically correct HTML structure.
Additionally, there is no <main> tag used for the main content. Including <main> around the primary section of the page would improve both accessibility and the organization of the HTML.
Using Prettier to format the code will help you maintain a cleaner and more consistent structure. This will ensure proper indentation, spacing, and overall readability.
Marked as helpful
1
