@kxnzx
Posted
Hello @Maher0551071612,
Well done finishing your second challenge! I have some suggestions...
-
You need to wrap the title "Gabrielle Essence Eau De Parfum" in a <h1></h1> tag, because this is the most important heading on the page. This helps the search engines understand what the page is about. Besides that it helps (screen)readers understand the hierarchy of the page. Always first start with an H1 and then you can follow it up with an H2 etc. (in case you have subheadings.) You could wrap "perfume" in an <h2></h2> tag. Even though this starts earlier on the page, this does not matter: the screenreaders will understand the hierarchy.
-
You are also missing the <main></main> tag. (Wrap or make your div in(to) a main tag). The main tag acts like a landmark. Screen readers and other assistive technologies read them. By using landmark elements such as <main>, you improve the navigation experience for users of assistive technology such as people with bad eyesight. The footer is also a landmark, and you included that. Well done!
-
Also..Look into CSS Flex-box and CSS-Grid to learn how to center containers on the page.
Marked as helpful