QR code first solution

Community feedback

• 

  RemyJuanes•90

  @RemyJuanesPosted about 1 month ago

  HTML

  • The white container doesn't have any border radius
  • Be careful, you use many div. Try to avoid it as possible as you can: it's a tag dedicated to divide content. I suggest to use semantic tags, in your code you can use <main> to put your "main" content. And <figure> for the picture then <figcaption> for its description.

  Semantic HTML increases the readability of your code and its maintainability.

  CSS

  • In your :root you can also add the lengths for padding, margin, ...

  Marked as helpful

  0