Skip to content
Submitted over 2 years ago

Profile card component

sushil540•40
@sushil540
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Profile card component
  • HTML
  • CSS
1newbie

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
    @iprinceroyyPosted over 2 years ago

    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
    @sushil540Posted over 2 years ago

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

    0

Please log in to post a comment

Log in with GitHub

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