Submitted 7 months ago

Social Links Profile Solution

Stanley Owarieta•60

@Stanley-24

Preview site View code

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'm proud to be part of this challenge, as it has allowed me to stretch my skills and creativity while learning new techniques and approaches.

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

--Initially, I faced challenges in determining the project's direction and completing it effectively. This uncertainty led me to delay work on the project for a day. However, I overcame this obstacle by seeking inspiration from a colleague's website. Although I didn't copy their code, I drew inspiration from it to understand the project better and formulate my approach. This approach enabled me to gain clarity and proceed with confidence in completing the project successfully.

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

--It would be greatly appreciated if anyone could assist in identifying any errors within the project and provide guidance on how to rectify them. Any insights or suggestions for improvement in specific areas of the project would also be valuable and welcomed.

Community feedback

yas-avocad•380
@yas-avocadPosted 7 months ago
Hey! It looks like you set your image to max-width: 100%, but you should set the height and width:

height: 75px width: 75px (or whatever the pixel amount is).

Also, for the buttons on the design, you have them as <div>, but you can just use a <button> element. I think it's easier to write the HTML and set the CSS that way.

Hope this helps :)
0
Stanley Owarieta•60
@Stanley-24Posted 7 months ago
@yas-avocad Wow, thank you so much for taking the time to check out my project and for providing such detailed feedback. I appreciate it! 🙏

You're right about setting the height and width of the image. I've gone ahead and made those adjustments as you suggested. Switching the buttons to <button> elements makes sense! It's going to make the HTML cleaner and easier to manage.

I've implemented your suggestions, so if you get a chance, could you please take another look and let me know if everything looks good now?

Thanks again for your help and support!
0
yas-avocad•380
@yas-avocadPosted 7 months ago
@Stanley-24 Yeah! Looks much better now. Nice job :) Also, for next time, try to follow the design exactly, because it's easier to see where there are differences. (So here, you have 6 buttons, instead of 5).
0
Stanley Owarieta•60
@Stanley-24Posted 7 months ago
@yas-avocad What do you suggest, should I remove one button?
0
yas-avocad•380
@yas-avocadPosted 7 months ago
@Stanley-24 I think you could move on to the next challenge :) Then try doing this one again when you have some more practice.
0

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