I used the html and Css but I couldn't find a free Icon.
Design comparison
Solution retrospective
I couldn't find free icons.
Community feedback
- @kxnzxPosted over 2 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.
Marked as helpful0 -
- @Luis15HerrPosted over 2 years ago
The cart icon is in the ./images folder.
Marked as helpful0 - @Sahitya2006Posted over 2 years ago
Hey, Your project is nice. You can use free icons from Fontawesome.com https://fontawesome.com/
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord