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

Hamā€¢ 340

@hmac100

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


What are you most proud of, and what would you do differently next time?

Was happy to understand and use css variables ( not sure if that wasn't overkill for a project that small though ? ) and getting the background svgs to display as shown.

What challenges did you encounter, and how did you overcome them?

The background needed a bit of research to understand how to do it but not too tricky or hard get info on. Initially the background was a mess as I was using percentages to position the svgs so the spheres moved when resizing - not a good look. The solution was using the key words right, left, bottom, top along with viewport width declarations in the background-position property.

What specific areas of your project would you like help with?

I think using more semantic html and also keen to fully understand and use bem mark-up . It seems a much better way of doing things.

Community feedback

Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello there!

Congrats on completing the challenge! āœ…

Your solution is really impressive!

I've got a couple of ideas (about how to use HTML better) that could make it even stronger:

šŸ“Œ First: Think about using <main> to wrap your main content instead of <div>.

Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

šŸ“Œ Second: Consider usinng HTML Headings (<h1> to <h6>) for your main title instead of <div>.

It's more than just text size ā€” it's about structuring your content effectively:

  • The <h1> to <h6> tags are used to define HTML headings.
  • <h1> is for the most important heading.
  • <h6> is for the least important heading.
  • Stick to just one <h1> per page ā€“ it should be the main title for the whole page.
  • And don't skip heading levels ā€“ start with <h1>, then use <h2>, and so on.

These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

Hope that's helpful!

Keep up the great work!

Marked as helpful

0

Hamā€¢ 340

@hmac100

Posted

@danielmrz-dev Once again thank you for those very useful tips : )

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