Skip to content
Submitted 5 months ago

Social Links Profile

jubileelinā€¢100
@jubileelin
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social links profile
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I tried to apply some knowledge that I learned from the last challenge about using different CSS units in order to create responsive designs and apply it to this design, such as using rem instead of px.

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

It was difficult getting used to rem, but I did realize how much more effective it is at creating responsive web design.

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

I want to know if I used rem correctly, I kinda just replaced all instances of px with rem and used a media query to resize the container on a mobile phone screen. I also would like to try and use mobile-first workflow on the next project I do, so if anyone has advice or tips about this concept, it would be appreciated.

Community feedback

  • Daniel šŸ›øā€¢44,310
    @danielmrz-devPosted 5 months ago

    Hey there!

    Congrats on finishing the challenge! āœ…

    Your solution looks awesome!

    šŸ“Œ It's a good idea to use semantic HTML elements like <ul> and <li> for lists. This makes your code more accessible, maintainable, and meaningful.

    Here's an example of 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>

    Using <ul> and <li> makes your content structure clearer, which is better for screen readers and search engines. Plus, it follows best practices for HTML.

    Hope this helps!

    Keep up the great work!

    Marked as helpful
    0
    jubileelinā€¢100
    @jubileelinPosted 5 months ago

    got it, will keep that in mind for next project @danielmrz-dev

    1
  • Kevin Toanā€¢60
    @CookielovekevinPosted 5 months ago

    I believe you are using rem properly, besides the one time you used it to set the border-radius. I think for the most part rem is used for fonts and font styling, whereas things like borders, outlines, margins should be px since it doesn't scale with zoom-in. (for the most part when you zoom in people want the text to be bigger not the boxes and margins they are in)

    Marked as helpful
    0
    jubileelinā€¢100
    @jubileelinPosted 5 months ago

    thank you, this was very helpful! @Cookielovekevin

    0

Please log in to post a comment

Log in with GitHub

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