Design comparison
Solution retrospective
Thanks to Marzuk Sanni for the advice. I was able to start building from the mobile version, then made it responsive for the desktop version. Any feedback is highly appreciated, thank you.
Community feedback
- @quielLovesLasagnaPosted 10 months ago
Great job, Tolulope Oluwabukunmi! Here are some suggestions:
- I can see that you did:
<h1>Jessica Randall</h1> <h3>London, United Kingdom</h3>
Here are some things that you must keep in mind when using headings:
- You should only have 1 <h1> element in your document. (main heading/main topic)
- You should not have a secondary heading <h2> if you don't have a main heading <h1>
- You should not have <h3> if you don't have <h2>
- You should not have <h4> if you don't have <h3> and so on...
Changing your code from the code above to the code below is the correct way of marking it:
<h1>Jessica Randall</h1> <h2>London, United Kingdom</h2>
- I can also see that you did this:
<section class="button-container"> <button>GitHub</button> <button>Frontend Mentor</button> <button>LinkedIn</button> <button>Twitter</button> <button>Instagram</button> </section>
this part of the profile is a list of social media platforms. using a ul and li's is the correct way of marking it:
<ul class="button-container"> <li>GitHub</li> <li>Frontend Mentor</li> <li>LinkedIn</li> <li>Twitter</li> <li>Instagram</li> </ul>
- Inside your external CSS file, I can see that you did not reset your CSS although this is "optional" (?) most/every developer does this so I suggest you do it as well. You can copy my CSS reset if you want to:
*, *::before, *::after { margin: 0; padding: 0; box-sizing: border-box; } img, svg{ mix-width: 100%; display: block; }
You can read more about this here or look it up online.
That is all. I hope these suggestions may help you.
Marked as helpful0@TheTreeveloperPosted 10 months agoWow, thank you so much! This was really helpful and insightful, I really appreciate the feedback and this will definitely help me go a long way to improving my code. @quielLovesLasagna
0@quielLovesLasagnaPosted 10 months ago@ioddgamers You're welcome! I'm glad to hear that!
0
Please log in to post a comment
Log in with GitHubJoin 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