social media

Community feedback

• 

  Tharun Raj•1,330

  @Code-BeakerPosted about 1 month ago

  Hi 👋 Good job completing the challenge. I want to share some of my suggestions for improving your solution.

  • Use colors with good contrast to make it readable. The links are not clearly readable because it's black inside grey.
  • You don't need to use all that unnecessary JavaScript in this project. Simply use the <a tag to create all those links. Something like this:

  <ul>
  <li><a href="#">GitHub</a></li>
  <li><a href="#">Twitter</a></li>
  ...
  </ul>
  

  • You could minimize the use of br tags by using a display: flex on the ul element and adding a gap: 1rem.
  • There's a typo in the project title: "Social medai" -> media
  • Make sure heading tags decrease one by one.

  h1
  h2
  h3
  ...
  ...
  

  This is essential for the accessibility of the website.

  • Instead of px, you can use either rem or em to make responsive layouts easier for you.
  • Use class selectors instead of tag selectors. Tag selection can mess up your code when you're dealing with large projects.
  • Use CSS Variables or Custom properties to make your CSS reusable.

  I hope this helps you learn something new! 🤝

  0