Profile card component

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 11 months ago

  Hello there 👋. Good job on completing the challenge !

  I have other suggestions about your code that might interest you.

  • If the image is decorative and doesn't convey meaningful content to the page, you can leave its alt attribute empty. This helps screen readers skip unnecessary information. Additionally, the alt attribute should describe the image, avoiding redundant words like "image", 'picture", or "photo" since the <img> tag already conveys that information. For a photo of a person, using their name as the alt text is a good practice.

  • You should use only one <h1> tag per page. The <h1> tag is the most important heading tag, This can confuse screen reader users and search engines. You can read more about this here 📘.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  0
• 

  Adrian Maj•170

  @AdrianMajPosted 11 months ago

  I think everything looks good there, it is accessible and responsive. You are using semantic HTML, layout looks good on phones. I have no reservations for that. Thanks!

  0