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

your-profile-card

@RohitPatra-2002

Desktop design screenshot for the Profile card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@rohitd99

Posted

Hello Rohit, from Rohit 😁

Congrats for completing the challenge.

I have but a few suggestions for you.

  1. You have used article as semantic element correctly so I'd recommend using main too instead of the div class="container" since the card is the main content of this page.
  2. Secondly I see you've used div for the background image I'd say you can use background-image property on your body for the same, since those images don't have any meaning this should be better. Here's a video from Kevin Powell that shows you how to use multiple images as background.Video

Hope it helps

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