Profile card component using flexbox (CSS)

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 months ago

  Olá Pablo, parabéns pelo seu projeto, ele está bem feito e se assemelha ao design proposto.

  Por algum motivo quando se clica no "Preview Site", ele abre uma página 404 do Github, você pode corrigir isso editando essa solução e passando a nova URL.

  Gostaria de dar algumas dicas. Sempre quando for utilizar fontes, tente utilizar a unidade de medida rem, pois ela se adapta ao tamanho de fonte configurada pelo usuário. Já a px é absoluta e não altera o tamanho baseado nessa configuração.

  Agora sobre o HTML, a sua estrutura não está errada, muito pelo contrário, mas notei que há pouca semântica e vou mostrar alguns trechos aqui.

  • <div class="card-perfil-container"> esse trecho poderia ser tanto uma <main> quanto um <article>, isso porque o card é tanto o conteúdo principal, quanto também ele faz sentido por si só.
  • <img src="images/image-victor.jpg" alt=""> o problema desse não está na tag de <img/> mas sim no <figure> que contêm essa tag. O correto era não ter o <figure>, isso porque utilizamos o <figure> para imagens com conteúdo mais significativo.
  • <section class="statistic-container"> já essa sua section não está errada, mas acredito que o que mais combina possa ser uma <ul>, isso porque os itens são todos relacionados entre si.

  Edit: A página já está carregando normalmente quando vamos no "Preview Site".

  E é isso, parabéns pela conclusão do projeto, ele ficou muito bem feito e se assemelha muito ao design proposto. Caso algo que eu tenha dito não tenha ficado claro, por favor, comente abaixo que tentarei ajudar da melhor forma possível.

  Marked as helpful

  1

  ---

  

  Pablo Vasconcelos•60

  @Pablo321123Posted 7 months ago

  @R3ygoski Muito obrigado pela profunda analise, Bernado! Ficou muito clara e com certeza tem muita valia para mim! Além disso, você solucionou muitas das minhas dúvidas para os próximos projetos. Portanto, só tenho a agradecer!!

  1