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

stats preview card component with flexbox and mobile first approach

@jlmcabral

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feel free to leave some comments 😉👍🦄

Community feedback

CyrusKabir 1,885

@CyrusKabir

Posted

hello my dear friend ♥ you card at all it's good and clean but here a problem :

  • don't use here a empty div tag for image and try to use a img tag it's more better and if you change the sizes you probably notice you picture get cut i mean you should look to understand

Marked as helpful

0

@jlmcabral

Posted

@CyrusKabir Thanks for the inputs!

I will change the empty div with an img tag.

About the image cutting, I'm not 100% sure if I understood the issue, but I will give it a look.

0
CyrusKabir 1,885

@CyrusKabir

Posted

@jlmcabral your welcome my dear friend notice me if you didn't understand the issue in image cut ♥

0

@thomashertog

Posted

what @cyruskabir already said about the image. Other than that. think about your HTML structure. right now you have headings of level 2 (h2 elements) with the numbers of the stats. Imagine you're browsing the webpage with a screenreader without actually looking at the screen and tabbing through the headings. It probably feels a bit weird that you are just announcing the amounts (and i don't really know whether it would be useful to have these as header at all)

0

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