Skip to content
Submitted over 2 years ago

Box sizing, import statements, margin, padding, font

David Burgess•110
@DavidBurgess1984
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'd like someone to review this code please. I'm particularly interested in best practices . What is the best way to use em for font-size? I did hard code the paragraph size to 15px. I'm trying to up my front-end css game.

Community feedback

  • BADHRI KESAVA RAJA S M•515
    @BadhrikrPosted over 2 years ago

    Hello your solution is superb,

    My only suggestion is ,Apply these to your stylesheet

    body{
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
margin: 0;
}

    These styles will help your content to be at the centre of the page.

    Happy coding.

    4

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