Yessica-jsy
@Yessica-jsyAll comments
- @mk-374Submitted 3 months ago@Yessica-jsyPosted 3 months ago
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 - @WrajtSubmitted 3 months ago@Yessica-jsyPosted 3 months ago
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 helpful1