@R3ygoski
Posted
Olá Keyla, parabéns pela conclusão do seu projeto ele ficou muito próximo do design proposto, parabéns.
Gostaria de começar falando sobre o problema que você teve, o arquivo não estava junto ao projeto? Isso é estranho, acabei de baixar o projeto base e ele veio com todos os arquivos necessários, esse é o diretório para a imagem: assets/images/illustration-article.svg, caso realmente não tenha isso no seu arquivo base, então não sei o que pode ter ocorrido.
Agora sobre a parte de duvidas, você pediu uma revisão completa do código. Sobre o CSS a única recomendação que eu teria seria de utilizar a unidade de medida rem no tamanho das fontes, pois dessa forma você faz com que sua fonte seja ajustada para as configurações de fonte do usuário. E também uma pequena correção, que no título "HTML e CSS foundations", ele é preto, ele só fica amarelo quando passa o mouse em cima.
Sobre o HTML, ele está bem estruturado, mas não está muito semântico, por exemplo:
<div class="imagem">poderia ser alterado para um<figure>.<div class="informacoes">poderia ser alterado para uma<section>.<div class="usuario">poderia ser alterado para um<footer>.
Ainda sobre acessibilidade, é sempre interessante ter uma tag <h1> por página, tanto que o título do card poderia ser esse <h1>.
Agora um pouco sobre a estrutura do HTML, por mais que ela esteja boa, não era necessário utilizar as seguintes div <div class="imagem"> e <div class="informacoes">. Você conseguiria ter feito todo o projeto sem utiliza-las, isso porque todos elementos dentro do card com exceção daquela parte do usuário, segue um fluxo padrão, isso é, de cima pra baixo, daí você poderia utilizar tanto margin quanto padding para aplicar o espaçamento correto. Mas claro, se você só remover eles sem fazer as devidas alterações, irá ficar de uma forma não esperada.
De resto é isso, novamente parabéns pela conclusão do projeto, ele ficou muito bem feito. Caso tenha ficado alguma duvida quanto eu que eu disse, por favor, comente abaixo que tentarei ajudar da melhor forma possível.
Marked as helpful
@KeylaLins
Posted
@R3ygoski, irei fazer os ajustes conforme sugerido. Sobre as imagens, realmente não tem entre os arquivos que baixei, mas vou tentar baixar novamente para verificar. Muito obrigada pelo retorno =)
