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

Sass

@vickymarz

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


I just completed this project. I need feedback on how to improve my code.

Community feedback

@pikapikamart

Posted

Hey, great work you got there. One thing to note upon, I think you font did not rendered properly? It is different compared to the original layout. Might want to check that one out.

Also while you're at it, you might want to change the declaration of the body tag's height from height: 100vh to height: auto and just set the min-height: 100vh. This way, your body tag will not be cut from the viewport especially when it is looked at dev tools.

Apart from that, good work^

1

@vickymarz

Posted

@pikamart thank you so much I have done the necessary corrections so you can check for confirmation.

0

@pikapikamart

Posted

@vickymarz Awesome, works well and thank you for considering my suggestion :>

0

@Will-1-Am

Posted

Have a look at the positioning of the avatar and the background pattern at the top of the card.

0

@vickymarz

Posted

Ok...can I know what you observed? 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