All comments

• 

  James Morrissey Jr• 30

  @JammiM

  Submitted 16 days ago

  

  Flexbox

  2

  30

  

  AntoineMu• 30

  @AntoineMu

  Posted 16 days ago

  Hi James,

  I really like how close you got to the suggested design, as well as your use of the @media query. I see you used flexbox on the body element to center your component horizontally. You can use align-items: center; on the body to center your element vertically. If you do this, you don't need to set a top margin on your qr-code-component. Also, if you set the you are going to use your font for every element, you can set it for the body element and all other elements will inherit the font. By doing this you don't have to set the font-family for every element.

  Marked as helpful

  1

  This comment has received 1 replyShow