Skip to content
Submitted over 1 year ago

Profile card solution using HTML, Sass.

C4rlos•410
@solracss
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

This one looked easy but wasn't :D These two bubbles was a hard battle. Finally after 2 days I've managed to finish it.

Also I encourage you to leave any comments regarding my code as it is win-win.

You practice code-review and I get feedback!

My process is described in my repo on GH. Check it if you are intrested how I've done it

Cheers!

Community feedback

  • Vanza Setia•27,775
    @vanzasetiaPosted over 1 year ago

    Hi, C4rlos! 👋

    One suggestion is to remove the "Picture of" phrases from the alternative text. Alternative text for images should not contain any words related to "image" (such as picture, photo, logo, icon, graphic, avatar). It is already an image element (<img>) so the screen reader will pronounce it as an image.

    Everything else on the HTML looks good to me. Great job!

    Marked as helpful
    0
    C4rlos•410
    @solracssPosted over 1 year ago

    @vanzasetia Will remeber this for future. Great that you looked at my solution. I really appreciate this. Thank you!

    0
    Vanza Setia•27,775
    @vanzasetiaPosted over 1 year ago

    @solracss You are welcome! 👍

    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