@benjoquilario
Posted
Hi GuidiUZ, great job on finishing this one, it looks good and response rather well.
some suggestion to improve your code;
- You need to change your
<input />
button into<button>
tag. The difference is that <button> can have content, whereas <input> cannot (it is a null element). - Instead of using
<img>
elements to wrapping your.logo
,.logo-empress
,.socials
and.footer-nav
you should use anchor element<a>
because this element is going to transfer user somwhere. - Considering adding a alt attribute on your images. Read more why we need to add alt
Good luck with that, have fun coding!
cheers, benjo
Marked as helpful
@GuidiUZ
Posted
@benjoquilario Thank for your time. I will implement what you have recommended. <3