CSS QR Code Component v2

Community feedback

• 

  Clément Creusat•1,655

  @ccreusatPosted over 2 years ago

  Hello !

  Nice job! Seems ok to me :)

  Some improvements :

  • You can replace your <div class="title"> by a <h1>, this will fix your acc. issue
  • Check the design if you can have a bigger line-height for the text.
  • You could change how to handle margin on title and description as well: -- For instance, you can remove all margin on .title and .description -- Add a padding: 24px on your .text -- Add this to your code :

  .title + .description {
      margin-top: 24px;
  }
  

  Marked as helpful

  0

  ---

  

  Paul THEUREAU•120

  @Paul21777Posted over 2 years ago

  @ccreusat Hello !

  I have done everything that you suggested. Have to fix things so that it exactly correspond to the design

  Thanks for the feedback !

  1

  

  Clément Creusat•1,655

  @ccreusatPosted over 2 years ago

  @Paul21777 Looks awesome man ! :)

  0