Skip to content
Submitted almost 4 years ago

Standard HTML/CSS. Tried to adhere to BEM. Probably too many DIVs....

5toCode•40
@5toCode
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

I really struggle with CSS...trying to get better. Feedback is welcomed and appreciated!

Community feedback

  • Jeury Santos•115
    @jeurysantos1Posted almost 4 years ago

    I really like how easy was to understand your code. One thing that I would do differently is not to insert the Pattern Images into your CSS, instead of into your HTML so you could position them in the right place in every single break Points.

    As well, you should have used the benefits of Media query in your CSS that would make the layout or design more responsive and won't avoid the design to break.

    0
    5toCode•40
    @5toCodePosted almost 4 years ago

    @jeurysantos1 thank you for the feedback, very helpful!

    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