Skip to content
Submitted almost 3 years ago

Responsive page with flexbox

Uririoghene Utobivbiā€¢70
@Ronarexx
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

Hello everyone! i have been off programming for a while but i am back now. All feedbacks will be appreciated. I used flexbox for this, what else would you have use instead of flexbox and how differently would you have built this.

Community feedback

  • Account deleted

    Hi there šŸ‘‹

    Congratulate on finishing your project šŸŽ‰. You did a great job šŸ”Ø

    I give some suggestions that I hope help you take your project to the next level šŸ“ˆ.

    1. Flexbox is great since it's just one-dimensional šŸ‘
    2. I think the card is too small you can increase the size also

    Happy coding ā˜•

    Maqsud

    1
    Uririoghene Utobivbiā€¢70
    @RonarexxPosted almost 3 years ago

    @maqsudtolipov thanks for the feedback, working on that asap

    1
  • ronaldseaā€¢40
    @RonaldSeaPosted almost 3 years ago

    Hello

    I recommend you use semantic tags because in most you used div's āœ… <section> for wrapping your component āœ… <img> for the image āš  <div> for bold text, here you can use a <h1> or <p>

    Keep improving. I congratulate you on your work!

    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