All comments

• 

  Semjon Nirmann• 70

  @semnim

  Submitted about 1 year ago

  

  Basic QR Code component

  1

  70

  

  Leonardo Gagliardo• 50

  @LeonardoGagliardo

  Posted about 1 year ago

  Hey Semjon, you did a nice job in this challange! But here's some advices that might help you improve this project or other's in the future.

  In the css global configuration you used:

  • { padding: 0; margin: 0; } Try adding:" box-sizing: border-box; " next time, it get's easier to control elements like border's, padding width and height. Look up to this configuration on ChatGPT for a full explanation.

  Also, you could have used h1 {margin-bottom: 15px } or p {margin-top:15px} to space the text from the title.

  Hope you find some of my advices useful :). And i'm sorry if there is some misspelling. I'm from Brazil and english is not my mother language.

  Marked as helpful

  1