Skip to content
Submitted over 2 years ago

CSS FLEX

Praise•80
@Omoh123
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I would love some feedbacks on properties i did not make use of well. Thank you

Community feedback

  • Saul•1,260
    @xsaulPosted over 2 years ago

    Hello, Praise I did a few changes to improve your project. To the image-text <div>, I changed the padding to 12px and the width to 40% like that the card will look wider.

    To the image, I changed the width and height to 100% so it can fit better in the container and look bigger.

    To the paragraph, I added a padding of 0 for top and bottom and 1rem for left and right like that the text will be centered.

    And don't forget that you have to use a specific font!

    Check out the values I told you and tell me what you think!

    0
    Praise•80
    @Omoh123Posted over 2 years ago

    @xsaul I just checked out the values and it looks so much better now. Thanks so much

    0

Please log in to post a comment

Log in with GitHub

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