Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Solution of the Blog Preview Card challenge

@AnaBeatrizA-Silva

Desktop design screenshot for the Blog preview card coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I am proud of the whole project, since again i manage to use renposivi units

What challenges did you encounter, and how did you overcome them?

I didn't have much difficulty with this project

What specific areas of your project would you like help with?

Maybe fitting the last text on the card

Community feedback

P. Ricardo 2,370

@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 classe card, 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

1

@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.

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord