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 link using CSS grid

Edperā€¢ 40

@edper

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?

Changed the color of the social links to its brand color or at least close to that one.

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

Not much but it's fun.

Community feedback

Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hey @edper!

Your solution is really impressive!

I've got an idea (about how to use HTML better) that could make it even stronger:

šŸ“Œ Consider employing <h1> to <h6> tags for headings rather than using <div> or <span>.

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.

When it comes to headings/titles, HTML offers us a range of heading tags: from <h1> to <h6>.

Here's a quick guide on how to use them:

  • The <h1> to <h6> tags are used to define HTML headings.
  • <h1> is for the most important heading.
  • <h6> is for the least important heading.
  • Stick to just one <h1> per page ā€“ it should be the main title for the whole page.
  • And don't skip heading levels ā€“ start with <h1>, then use <h2>, and so on.

It's more than just text size ā€” it's about structuring your content effectively.

These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

Hope that's helpful!

Keep up the great work!

1

Edperā€¢ 40

@edper

Posted

@danielmrz-dev Thanks for the feedback. Appreciate it.

0

@Brian-Pob

Posted

Hi Edper! Good job on the solution! The customizations look great.

I do have a few recommendations

  • Use a ul tag with li tags for the links since you have a list of items. It helps screen readers dictate that the user will encounter a group of elements.
  • Try exploring options that let the a tag take up the whole size of the li. This way, you won't encounter small sections of the element that are not clickable such as the icons in your solution.

Hope this helps!

1

Edperā€¢ 40

@edper

Posted

@Brian-Pob That's a good idea thanks for the feedback brian.

1

@Brian-Pob

Posted

@edper No problem! Don't forget to kindly mark the comment as helpful if you thought it was useful! šŸ˜

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