@Saekit
Posted
This solution looks good and matches the design well. Well done! I have a couple suggestions for improvement:
- Since the social links are technically navigation buttons it would be better to use an
<a>
tag or a<button>
tag instead of a<p>
within an<li>
. Using a button tag will also allow the user to select the button using the tab key. - Some of the formatting, like in
SocialLinks.jsx
, is a bit off. Installing prettier in your code editor can help with that. Your Tailwind and reusable components look good!
Marked as helpful