@pRicard0
Posted
O projeto já está perfeito, sem erro de acessibilidade, fazendo uso de boas práticas e tudo... mas vou fazer algumas recomendações do que está me incomodando um pouco. É apenas minha opinião.
-
Acho que ao invés de usar a tag
<section>
com a classecard
, acho que seria melhor usar a tag<main>
. No fim não faz tanta diferença de acessibilidade já que main se torna mais relevante quando há a presença de outros conteúdos mas... acho que mesmo assim, main se encaixaria melhor e previne de você usar section errado em algum momento. (Isso também serve para o seu projeto do QR Code) -
Recomendo que aumente o line-height do paragrafo abaixo do título. Não sei se você deixou desse jeito por gosto pessoal mas independentemente, aumentando um pouco mais o espaçamento melhora a leitura e é o recomendado. No livro Refactoring UI, os mesmos criadores do framework TailwindCSS, falam sobre isso. Para linhas curtas e texto pequeno, line-height de 1.5 é o recomendado ( como o seu texto é maior, talvez uns 1.3 - 1.4, fica ao seu critério ). Você pode ler um pouco sobre aqui https://www.refactoringui.com/previews/line-height-is-proportional
-
Você escreveu "Publish" ao invés de "Published" kkkkk, mas isso tanto faz.
Marked as helpful
@AnaBeatrizA-Silva
Posted
@pRicard0
Eu tava usando o section justamente porque não tinha outras coisas na página, então eu não via sentido em usar o main. Mas obrigada pelo aviso.
Realmente, fui o olhar o meu projeto, e usar um line-height teria deixado o texto bem melhor.
Obrigado pelo aviso do "Published', meu inglês escrito não é muito bom, mas estou tentando melhorar.