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

Mobile first approch, CSS Grid and Flexbox, semantic HTML

@Hubnotch

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


Please criticize and give your honest feed back. Where do you think I need to improve on?

Community feedback

Ahmed Faisal• 5,095

@afrussel

Posted

Hi Ugwuanyi Love Ekene!

Thanks for your work. I found some issues with your work. These are as follows:

  • On the Desktop view card is not center both horizontally and vertically.
  • Background image is not fit like the mockup.
  • If you don't mind please take a look at this solution [link] (https://www.frontendmentor.io/solutions/frontend-mentor-profile-card-component-QQpAc6OOL)

Thanks in advance. Happy coding.

Marked as helpful

1
Vanza Setia• 27,855

@vanzasetia

Posted

👋Hi Ugwuanyi Love Ekene!

I have some feedback for this solution:

  • On my mobile device the card doesn't on the center of the page.
  • You can write the name of the person in the alt attribute.
  • For CSS, you don't need to mention the HTML tag, like div, section and etc. You can directly target the element using its class name.
  • Make sure that you have a consistent indentation when you're writing HTML and CSS.

That's it! Hopefully this is helpful!

Marked as helpful

1

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