👩‍💻 QR Code Component

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  👾Oi Carolina, tudo bem?

  Parabéns pela sua solução está muito bem feita, gostei do que você fez porque está responsiva inclusive em telas muito estreitas, mandou bem!

  Minha dica para você seria tirar a div chamada info, nesse caso nem teria a necessidade, só com a <main> você já consegue segurar todo conteúdo e além disso você já definiu o padding pra separar os elementos na main, então ficou perfeito, pra dar uma limpada no seu código seria uma tirar a div info.

  Vi que você usou rem e isso é bacana, pouca gente foca nisso, não sei se você sabe, mas no VSCode tem um plugin que faz a conversão em automatico, chama px to rem dai nem precisa ter o trabalho de converter todos px por 16, o plugin converte tudo no final do projeto.

  Parabéns, sua solução ficou muito boa, pouca gente acerta todos detalhes já nos primeiros desafios.

  Marked as helpful

  1

  ---

  

  Caroline Amorim•50

  @CarolAmorimPosted over 2 years ago

  @correlucas Oi, Lucas!!

  Muito obrigada pelo feedback, irei aplicar sua dica ao código. E a respeito do plugin eu não o conhecia, vou começar a utilizá-lo, vai me ajudar muito.

  1

  

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  @CarolAmorim fico feliz em saber que essa informação te ajudou de alguma forma, o plugin vai ser ótimo, só aplicar no código no final de tudo, eu perdia mto tempo mudando todos os pixels do código, com esse plugin eu seleciono todo o css e ele converte tudo 😂 tenha um bom dia Caroline.

  1