Social Links using CSS Hover

Community feedback

• 

  Fraol T.•60

  @Fraol-TPosted 2 days ago

  I checked your code and I noticed that you didn't use containers that much and that answers your struggle to center every element. I recommend you to section every element in a relatable group such as profile info and links that would make the code easy to use and understand. Also if you notice some elements structured as a list in the design it is wise to use an unordered list <ul>.

  Other than that, you developed a well done page. Keep up the good work 👍

  Marked as helpful

  0
• 

  Adriano•39,410

  @AdrianoEscarabotePosted 1 day ago

  Hi WajahatAli130, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

  Avoid using the <br> tag in your HTML code. While <br> might seem like a simple way to break lines, it is considered bad practice and can lead to significant accessibility concerns. For users who rely on screen readers, the presence of <br> can be announced, which disrupts the flow of the content and creates a confusing experience.

  Instead of <br>, you should use semantic HTML to structure your content properly. For example, wrapping text in paragraphs (<p>) or using <div> containers for sections provides a cleaner and more accessible solution. This approach improves usability for screen readers and ensures that your content is presented in a meaningful way to all users.

  For more detailed guidance, refer to the MDN documentation on the <br> tag: MDN: Accessibility Concerns of <br>

  Pro Tip: Accessible web development isn't just a recommendation—it's essential for ensuring inclusivity on the web!

  The rest is excellent.

  I hope you find it useful. 👍

  0