Card QrCode

Community feedback

• 

  Thiago Correa•350

  @ThiagoBRG60Posted 11 months ago

  Oi 👋!

  O seu QR code component ta muito bom em geral, mas aqui estão algumas sugestões:

  1 - Você incluiu uma tag <main>, isso é importante pra definir o conteúdo principal de um site, a única observação aqui é que você deixou os elementos "soltos" dentro do main, eu recomendaria usar a tag <section>, ou entao agrupar os elementos dentro de uma div, já que em muitos sites as vezes você pode ter muitos elementos, e deixar eles apenas na tag main sem agrupamento ficaria um pouco mais dificil depois pra dar manutenção.

  2 - Mesmo sendo um projeto simples, sempre adicione a descrição da imagem no alt"", isso é muito importante pra acessibilidade e pessoas usando leitores de tela.

  3 - Eu vi que pra centralizar o main voce usou o translate(-50%,-50%), você pode ter o mesmo resultado usando flexbox, inclusive é até mais fácil para centralizar os outros itens dentro do main, pois vi que você usou position absolute e aplicou margin nos elementos. E para dar um espaçamento interno, para afastar os itens da borda, você pode usar o padding.

  De geral, são essas sugestões mesmo, continue aprendendo e melhorando 🤝🏻.

  Marked as helpful

  1