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 finished

Spencerā€¢ 80

@Divinitry

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 are you most proud of, and what would you do differently next time?

Proud of the tiny details I added to make it look more unique

What challenges did you encounter, and how did you overcome them?

Problems with transitions and I overcame them by looking up tutorials and better understanding them with some research.

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

naming conventions with class names, anything people can find that I can improve on please let me know.

Community feedback

Borisā€¢ 2,870

@mkboris

Posted

Nice work @Divinitry, here are a few things to review

  • All content should be wrapped within landmarks. Wrap a main tag around card. and a footer for the attribution
  • Using an unordered list ul to group the social media links is a better approach for both semantics and accessibility.
  • It's best practice linking Google fonts directly in the HTML head section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times.
  • Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset.

Marked as helpful

2

Spencerā€¢ 80

@Divinitry

Posted

@mkboris

oh wow, thank you very much for the feedback, will put these to good use!

1
MikDra1ā€¢ 6,050

@MikDra1

Posted

Nice one, @Spencer,

Here are some tips from me:

  1. You should never make your body height 100vh. Instead you should try to make a container that will have this height.

  2. If you have a font-family always try to add one additional font, like sans-serif, at the end, so if the browser doesn't understand the imported font, it will use the second one.

  3. Next time, try to add a CSS reset to your code. If you don't know how to do it, here is a link for you -> POST and here is also a VIDEO

If you find this comment useful, please mark it as helpful šŸ’—

Good job, and keep going šŸ˜€šŸ˜šŸ˜‰

Marked as helpful

1

Spencerā€¢ 80

@Divinitry

Posted

@MikDra1

Thank you so much for this!

Another comment here mentioned CSS reset so I will for sure look into this. Do you mind expanding on the first point? What are the drawbacks of making my body 100vh? and instead switching it to a container, what would that accomplish? just wanna learn more, thanks!

0
MikDra1ā€¢ 6,050

@MikDra1

Posted

@Divinitry

Sorry for that I wasn't clear. I meant if you want to you can use height one body, but instead of fixed height you should you min-height so if the content will overflow we will still see it.

šŸ˜€šŸ˜šŸ˜Š

Marked as helpful

0
Dhruvā€¢ 210

@Dhruvmehta1311

Posted

Looks awesome.

1

Spencerā€¢ 80

@Divinitry

Posted

@Dhruvmehta1311

thanks!

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