Submitted 5 months ago

Social links profile

John McNichol• 120

@inappdesign

Preview site(opens in new window)View code(opens in new window)

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?

I'm happy I have done another Frontend Mentor challenge! I will aim to do another one this week and try to increase my output even further within the next month.

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

I wanted to go beyond what the challenge was asking, just something a little extra, so I have put icons and some minor animations on each button.

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

I've seperated my CSS with comments into different parts: CSS reset, Tags and Classes. I'm not sure if this makes it more readable but it was all getting a bit messy as I was building the challenge. Is there a standard way everyone does it?

Community feedback

Daniel 🛸• 44,230

@danielmrz-dev

Posted 5 months ago

Hello there!

Congrats on completing the challenge! ✅

Your solution looks great!

I have a suggestion for improvement:

📌 Think about using <main> to wrap your main content instead of <div>.

Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

This change might not have impact on how your page looks, but it'll make your HTML code clearer and help with SEO and accessibility.

Hope that's helpful!

Keep up the great work!

Marked as helpful

John McNichol• 120

@inappdesign

Posted 5 months ago

Thanks, @danielmrz-dev,

I will make this change on my next upload.

Please log in to post a comment

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