@Krista Calleja, Great job on responiveness, regarding the share icon on the mobile view I sugesst that you include the share icon in the class="footer__social-icons-overlay" and use some sort of modifier or a extra class plus some js to get the behavior.
I would really appreciate if you take the time to answer any of the following qestions.
- How the HTML could be improved ?
- This is my first time using BEM, do you see it being overused ? 3 How can you change the background color of share-icon to white on hover state ?
I would really appreciate it if you take the time to answer some of this questions.
- Do you see any redundency in the HTML if so how can it be improved ?
- Is there any batter way to do the icon-star if so how ? 3 The background images are not fully responsive how can they be made responsive ?
@Stephane Baroux Great job, it look nice, replacing the div with the class of main with a main tag makes you solution more sementic.