Projeto-qrcode

Community feedback

• 

  Pedro Henrique•20

  @PdroOlivePosted 6 months ago

  Vlw pelas visões estou ciente, porém obg pelo toque.

  meu intuito com o display:flex; no body era apenas para centralizar verticalmente sem precisar de paddings, nem me toquei realmente que poderia ter tirado o margin do main. e a questão do alt n costumo preencher realmente por desleixo kkk, sei que e errado. mais uma vez vlw pelo feedback meu amigo.

  0
• 

  P. Ricardo•2,370

  @pRicard0Posted 6 months ago

  Nada a declarar... brincadeira!

  Dicas HTML

  • Como você já estava utilizando flex na tag body para centralizar a main, seria mais intuitivo que você utilizasse justify-content: center; ao invés de um margin auto na main.
  • Imagens que não são meramente ilustrativas devem conter uma descrição no atributo alt. O atributo alt serve para você auxiliar pessoas com deficiências visuais, permitindo que o site forneça informações sobre o que está sendo apresentado visualmente usando os screen readers. O atributo alt é uma informação que será fornecida para a pessoa, para isso você precisa descrever o que é aquela imagem para que o screen reader repasse. Você pode por, por exemplo... "QR Code to Frontend Mentor"
  • Não é recomendado utilizar a tag <article> pois o conteúdo dentro dele não é independente, não funciona fora do contexto do site. É melhor usar <section>
  • Utilize <h1> ao invés de <p> para o texto "Improve your front-end skills (...)". Todo site deve conter um h1 por questão de acessibilidade. Não importa se o h1 fique invisível ou não, tem que ter.

  Dica CSS

  • Utilize a unidade de medida rem para font-size ao invés de usar px. Essa medida é mais flexível, adaptável, se molda com base nas preferências do usuário e etc... Você pode entender mais sobre aqui https://gabcarvalhogama.com.br/css/qual-unidade-de-font-size-usar-px-rem-em-vw/

  0