Responsive Article preview component

Please log in to post a comment
Log in with GitHubCommunity feedback
- P@AnjelToppo
You did a great work.
Regarding the
.footer-button-section
class, I observed that the content is enclosed within<button>
tags. Considering their likely function as links, would it be beneficial to update them to<a>
tags instead! For instance:BEFORE
<button class="footer-active-button"> <img src="images/icon-facebook.svg" alt="" /> </button>
AFTER
<a class="footer-active-button" href="/"> <img src="images/icon-facebook.svg" alt="" /> </a>
Join 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