
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Mohammed-Bata
Looks good. 1.It is links, so should be
<li><a href="#" >Github</a></li>. 2.to add cursor while hover ul li a:hover{ Pointer:cursor; } I hope you find that helpful. - @Toskytd
very detailed work...i think you failed to add your cursor type and color...otherwise it's brilliant work.
- @gautam-32b7
Hi there!
Your solution looks amazing. However, there are a few mistakes.
<ul class="profile__social-links"> <li class="profile__social-link">GitHub</li> </ul>
Use anchor tag. The
<a>
tag defines a hyperlink, which links from one page to another.<ul class="profile__social-links"> <li class="profile__social-link"><a href="#">GitHub</a></li> </ul>
Use
rem
overpx
. rem units are based on the document's root element (HTML) and are therefore more flexible and responsive than px units.html { font-size: 62.5%; }
I hope you find this helpful, and happy coding!
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