All comments

• 

  MizaelGbb•10

  @MizaelGbbSubmitted 8 months ago

  

  QR code component

  1

  30
  

  k1te•40

  @k1tesurfenPosted 8 months ago

  Apart form the size difference, you really hit the proportions. Important are the border-radii which should be parent border-radius minus the offset to the inner element. So here its something like 18px radius minus 10 px padding = 8px inner border radius.

  The drop-shadow is a little weak. I really find it hard with different monitors and contrast settings.

  0
• 

  ImprovedFaund•10

  @ImprovedFaundSubmitted about 1 year ago

  

  QR Code CSS Design

  1

  50
  

  k1te•40

  @k1tesurfenPosted about 1 year ago

  **First impression

  You can clearly see the design of the given jpg in your solution. The site works good for all screen sizes. Text placement and font weights are spot on!

  **What could be better?

  -Some sizes are a little off. Size of the card itself is too small and the padding of it as well. You could use some image editing software (Photoshop, Affinity Designer, Gimp) to measure all pixel values to make it closer to the desired design. -The footnote is not visible, only if you scroll down. You could just make it 'position: absolute' and stick it to the bottom with 'bottom: 10px; left: 10px'

  But congrats on the challenge. Keep it up

  Marked as helpful

  1