@benjoquilario
Posted
Hello, Shawky! ๐
Congrats on finishing another challenge! ๐ Your solution looks very good and also responds well.
If you are having problems in the features section you can check my work here the link
Here some suggestion:
- All page content should be contained by landmarks. It will better if the structure of your html page is like this
<header>
<nav></nav>
</header>
<main>
<section>
</main>
<footer></header>
_ You need to add the navigation link. Because this element is going to transfer user somewhere. <li><a>tag here</li>
and also best practices.
On Footer
- Instead of using
<img>
elements to wrapping your social container you should use anchor element<a>
because this element is going to transfer user somewhere. Eg:<a href="#">Social icon here </a>
Aside from those everything looks good to me. Keep coding and happy coding too!