Boris• 2,870
@mkboris
Posted
Hi @saifuldt nice work, here are some suggestions that might be of interest to you.
- Responsive images are handled using the
pictureelement. Using CSS to hide and show images to display differently on various screen sizes is not part of the best practices. - The logos are meant to be links to the home page , the way to handle them will be to place the logo image in an anchor element.
- Add aria-label for social links
<a href="#" aria-label="visit Facebook"><img/></a>
Hope this helps.
0
