Skip to content
Submitted 27 days ago

Flexbox

garnetcodes•60
@garnetcodes
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

What are you most proud of, and what would you do differently next time?

Proud of applying what i have learnt

What challenges did you encounter, and how did you overcome them?

Encountered problem with Flexbox but overcame it after several trials and error

What specific areas of your project would you like help with?

Flexbox

Community feedback

  • P
    Trischa-99•170
    @Trischa-99Posted 27 days ago

    Hi, Your solution looks great! Personally, I don’t see many areas that need improvement. However, you could consider using shorthand for padding and margin. While it’s not always necessary, it could enhance readability in some cases.

    .card {
    padding-top: 16px;
    padding-right: 16px;
    padding-bottom: 40px;
    padding-left: 16px;
}
-->
.card {
    padding: 16px 16px 40px 16px;
}
    1
    garnetcodes•60
    @garnetcodesPosted 26 days ago

    @Trischa-99 You are right. Thanks for pointing it out for me

    0
  • Toskytd•100
    @ToskytdPosted 27 days ago

    clear and readable...you could write some properties using the shorthand method(e.g padding: 10px 20px 30px 10px) to shorten the code...welldone

    0
    garnetcodes•60
    @garnetcodesPosted 26 days ago

    @Toskytd Thanks for pointing it out for me

    1
    garnetcodes•60
    @garnetcodesPosted 26 days ago

    @Toskytd Thanks for pointing it out for me

    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