Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Profile card component

P
eldmarโ€ข 160

@eldmar

Desktop design screenshot for the Profile card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feedback is very welcome!

Community feedback

@MelvinAguilar

Posted

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

@AdrianMaj

Posted

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

Please log in to post a comment

Log in with GitHub
Discord logo

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