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

HTML, CSS, CSS Flexbox

@DxnielXndrew

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


Pretty Simple. Colour may differ from solution

Community feedback

P
hardy 3,660

@hardy333

Posted

Hi Daniel, everything looks good and clean on Desktop and mobile screen sizes.

But on tablet/middle size there is bug where image is not positioned correctly.. maybe you missed it while codding :(

Overall very nicely don, good job.

Marked as helpful

4

@DxnielXndrew

Posted

@hardy333 no, that was sort of intentional. I usually leave everything in between 1440px and 375px as custom. But now that you bring it up, It would look better with the image on top

0
darryncodes 6,430

@darryncodes

Posted

Hi Daniel,

Great design, really responsive - well done!

Some thoughts from me:

  • add order: 2; to .text on mobile to swap the order of the image and text around
  • i'd add max-width: 1000px; to your .box class to stop the design from stretching on wider screens. The design and image starts to break on wide screens

All the best!

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