Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Links Profile

P

@hadeedji

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What specific areas of your project would you like help with?

I would love comments on html semantics.

Community feedback

P

@chelsea-here

Posted

Hi there, it looks like you are missing focus states. You can add it in a very similar manner to how you added hover states in tailwind.

< a className="focus:bg-green focus:text-gray">text here</a>

Regarding your html the only thing I'm not clear on is why you would add an div below your body and above your main element. Otherwise, looks good to me. Cheers!

1

P

@hadeedji

Posted

@chelsea-here Thanks for feedback. I always forget the focus states, wish there was a way to have the same hover and focus styles, because in most cases you want the same ones, is that correct?

About the div, I used to always surround everything by a div and use that for centering, bg etc, because I had this idea that I shouldn't add classes to the body element, but in a later challenge I did add classes to body. Would you say that's preferable?

0

Please log in to post a comment

Log in with GitHub
Discord logo

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