Skip to content
Submitted almost 3 years ago

Responsive mobile-first profile card component

Noemi Gali•280
@NomiDomi
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

For this project I focused a lot on CSS variables and how to use them properly (check my README.md file for more info). I also kept my code as DRY as I could.

Not super happy with the way I dealt with the background images and couldn't find a solution that did it better. Any suggestions?

And as always, I appreciate any feedback! :)

Community feedback

  • Carl Wicker•1,055
    @carlwickerPosted almost 3 years ago

    Really great work, consider fixing those accessibility issues in the report.

    • There should only be one H1 on a page, so probably move that to the name.
    • Completely remove all the other header tags, consider spans or divs.

    Keep up the great work.

    Marked as helpful
    0
    Noemi Gali•280
    @NomiDomiPosted almost 3 years ago

    @carlwicker Thank you Carl for your feedback! :) Just implemented it!

    1
    Carl Wicker•1,055
    @carlwickerPosted almost 3 years ago

    @NomiDomi Awesome stuff :D

    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