Skip to content
Submitted over 1 year ago

CSS grid, CSS flexbox

Ferney•170
@FerneyMontoya6
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

If someone have some feedback I'd really like to know it 😊

Community feedback

  • Fer•3,970
    @fernandolapazPosted over 1 year ago

    Hi 👋🏻, some of this may interest you:

    • The trick of setting font size: 62.5% is no longer commonly recommended, and it is good to get used to thinking directly in relative units since they are the most suitable for most cases. More about the 62.5% trick here 📗
    • You should remove the media from <link rel="stylesheet" href="css/styles.css" media="screen and (min-width:375px)">. It is not necessary and it makes the page not have styles below that width.

    I hope it’s useful : )

    Regards,

    Marked as helpful
    1
    Ferney•170
    @FerneyMontoya6Posted over 1 year ago

    Thank you so much! @fernandolapaz I really appreciate your feedback 😊

    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