HTML, CSS, CSS Flexbox

Community feedback

• 

  hardy•3,660

  @hardy333Posted about 3 years ago

  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

  ---

  

  Daniel Andrew•110

  @DxnielXndrewPosted about 3 years ago

  @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,370

  @darryncodesPosted about 3 years ago

  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