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

HTML, CSS

@TejveerSinghRajput

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


Hello friends give me Any feedback is welcome.

Community feedback

Zuzki 180

@LuisDGracia

Posted

Hi, I see you made the design but there are some things you could do to improve, having using a position relative/absolute on the body is not the ideal solution since the more elements you add more work you'll need to do on the positioning, I suggest to you to learn about CSS flexbox and CSS grid that could improve a lot your CSS and reduce the lines of code you use.

I see you have "font-family: 400", font-family stays for the name of the font, like Arial, sans-serif and so, font-weight only accepts 1 value you can't use 400, 700.

using a table for the footer is not a bad idea, but not the ideal, you could encapsulate the 3 elements each one on their div container, and use a display flex yo put then on the center.

I hope this could help you to improve and remember to check the report to get more info about you could fix

1

@TejveerSinghRajput

Posted

@LuisDGracia Hi, Thanks for giving feedback and, I am new in front end developer. I will improve my solution.

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