Responsive Social links profile using flexbox

Hello there!

Congrats on completing the challenge! ✅

Your solution looks great!

📌 It's recommended to use semantic HTML elements like <ul> and <li> for creating lists. This ensures that your code is more accessible, maintainable, and semantically meaningful.

Here's and example on how you can refactor your code:

After Refactoring

<ul class="list-container">
    <li><a href="#">Github</a></li>
    <li><a href="#">Frontend Mentor</a></li>
    <li><a href="#">LinkedIn</a></li>
    ...
</ul>

By using <ul> and <li>, you convey the structure of your content more clearly, making it easier for screen readers and search engines to understand. Additionally, it aligns with best practices for HTML semantics.

I hope you find this helpful!

Keep up the excellent work!

Hi, congrats on completing this challenge! 🎉

I checked your repo and can give you a few comments on your solution.

Does the solution include semantic HTML?

• ✅ There is a main landmark
• 🔶 The avatar could use a better alt in my opinion. like the person name maybe?
• ✅ Good use of the bloquote element :)

The links is a list of links. You could wrap them in an <ul> and use <li> for each link.

Is it accessible, and what improvements could be made?

• ✅ relative units are used.
• The link are hardly readable when hovered or focus. I would update the text color to a dark color for these interaction states.

Does the layout look good on a range of screen sizes? yes.

Is the code well-structured, readable, and reusable? I would not add the css file in the /assets directory but keep it at the root of the folder in your current structure. Use the /assets for images and fonts.

You can use a modern css reset to make your life easier later with different spacing and sizing issue between browsers.

I suggest you use scoped classnames for the different elements it will make your code more readable and reusable. For example you don't want to change all the <p> tag to green but only some specific ones. Currently it makes it difficult to understand quickly if we want to do an update.

Does the solution differ considerably from the design?

There are some difference from the design and it looks smaller. To help you can use a tool like This perfectPixel chrome extension. You can use the file from the /design folder to compare to your actual solution.

I hope you find my comments helpful for improving your solution and your future challenges. Don't hesitate to reach out if you have any questions.

Happy coding !