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

sushil540 40

@sushil540

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


It took time to fulfill my satisfaction, but the outcome was fruitful. looking forward to update it's code so it look good on smaller screens, any correctness and feedbacks for making it responsive would be helpful. Thank You!.

Community feedback

Prince Roy 1,210

@iprinceroyy

Posted

That's excellent work @sushil540. You need to focus on some key points here:-

  1. There should be one heading tag on a web page, if there is no heading element requirement, then set its font size to 0. HTML- <h1></h1> CSS- h1{ font-size: 0; }
  2. The content should be wrapped inside the main landmark to avoid accessibility issues. Like this <body><main>your content goes here......</main></body>.
  3. Id should be unique. You change the other id.
  4. Don't set the width & height of an image inline, do it in a stylesheet img{ width: height: } Hope it adds to your learning. Happy coding :)
1

sushil540 40

@sushil540

Posted

@iprinceroyy Thank you so much for your valuable key points!.

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