Skip to content
Submitted almost 3 years ago

QR code component

AYUSH RANJAN•170
@aayusranjan
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

Need Improvement :

  • card shadow, and is it okay to add @improt more than one times or is there any other better option
  • I can't think about adding more necessary thing in CSS body tag.
  • class card using margin to do center ( is it good way )

Community feedback

  • Raymundo Alva•40
    @rayalva407Posted almost 3 years ago

    Hello Ayush! Really good solution! Keep up the good work. Instead of using margin to center the div I would recommend using flexbox. It is a lot more simpler to implement and cleaner code in my opinion. Here is a link to learn more about flexbox https://www.w3schools.com/csS/css3_flexbox.asp.

    Marked as helpful
    1
    AYUSH RANJAN•170
    @aayusranjanPosted almost 3 years ago

    Thank you for look on it, I will sure use it.

    1

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