I used the html and Css but I couldn't find a free Icon.

Community feedback

• 

  kxnzx•870

  @kxnzx

  Posted about 3 years ago

  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.

  00

  Marked as helpful
• 

  Luis Herrera•795

  @Luis15Herr

  Posted about 3 years ago

  The cart icon is in the ./images folder.

  00

  Marked as helpful
• 

  Kallakuri Sahitya•100

  @Sahitya2006

  Posted about 3 years ago

  Hey, Your project is nice. You can use free icons from Fontawesome.com https://fontawesome.com/

  00

  Marked as helpful