Responsive design with grid and flexbox

Community feedback

• 

  gfunk77•1,200

  @gfunk77Posted about 1 year ago

  Well done Alejandro. Your solution looks just like the picture so it is totally excellent.

  I would suggest experimenting with something like this:

  <main class="container"> <div class="wrapper"> <img src="./images/image-qr-code.png" alt="QR code" /> <h1>Improve your front-end skills by building projects</h1> <p> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level </p> </div> </main>

  All I did above was put the img, h1, and p inside the wrapper. From here, you could just put a padding all the way around on the .container. Put border: 2px solid color on the boxes so you can see what is happening and what is taking up space. Then I'll think you'll find that you wont need to put widths on the h1 and p. You won't even need the flexbox on the container. This might help streamline your css.

  I hope this is helpful feedback. Great work!!

  Marked as helpful

  1

  ---

  

  Alejandro Cañon•260

  @alecanonmPosted about 1 year ago

  Thank you so mucho! I’ll put it in practice @gfunk77

  0
• 

  Jewlius X•30

  @jewliasPosted about 1 year ago

  Great work Alejandro on completing the QR code challenge

  1

  ---

  

  Alejandro Cañon•260

  @alecanonmPosted about 1 year ago

  Thank you 👾 @jewlias

  0