Skip to content

projeto#18

Open
Piedrovi wants to merge 1 commit intomasterfrom
projeto-portfolio
Open

projeto#18
Piedrovi wants to merge 1 commit intomasterfrom
projeto-portfolio

Conversation

@Piedrovi
Copy link
Collaborator

nao ta completo falata mt coisa

@Piedrovi Piedrovi requested a review from alinelabenu May 28, 2022 02:44
@yMarceloMaia yMarceloMaia self-requested a review May 31, 2022 14:19
Copy link

@yMarceloMaia yMarceloMaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
O usuário deve ser capaz de encontrar uma descrição sobre a pessoa desenvolvedora
O usuário deve ser capaz de encontrar uma lista de instituições de ensino e empregadores prévios -
O usuário deve ser capaz de encontrar uma galeria de projetos de software que a pessoa desenvolvedora trabalhou -
Ao clicar em cada projeto da galeria de projetos deve abrir uma página nova com detalhes do projeto -
Cada página de projeto deve ter (pelo menos) uma imagem do projeto, um título, uma descrição e uma lista de tecnologias utilizadas -
O usuário deve ser capaz de encontrar uma lista de habilidades que a pessoa desenvolvedora possui (técnicas e comportamentais) -
O usuário deve ser capaz de encontrar um formulário para contato. Esse formulário deve receber nome, email e texto -
O usuário deve ser capaz de encontrar uma lista de redes sociais e respectivos links para contato -
O usuário deve ser capaz de encontrar um menu na parte superior da tela com links para as demais áreas do portfólio
Quão fácil foi achar as informações no site? Médio
O que você achou do design do site? Bom
Funcionalidade extra -
Feedback do código Feito
Nomenclatura de classes e ids fazem sentido -
Imagens são endereçadas corretamente usando caminho relativo
Boa indentação
CSS bem organizado
Tags HTML bem escolhidas
Responsabilidade de estilização é exclusiva ao CSS
Navegação com tag funciona como esperado -
Utilizou a propriedade alt da tag -
Arquivo principal se chama index.html
Criou link do surge
Criou um bom readme com informações relevantes para o projeto -

Comentários da pessoa avaliadora

Olá Pedro!

Parabéns pela entrega do projeto portfólio!

Alguns pontos sobre seu código!

  • Não sei dizer se foi sua intenção, mas ficou o maggin e padding padrão dos elementos no site, isso deixa uma borda branca ao redor do site, para resolver isso é só colocar no seu css
* {
   magging: 0;
   padding: 0;
}
  • Você poderia usar palavras mais informativas para definir o nome das class, tipo a class com nome de 'btt1' não é muito sugestivo do que se trata, isso pode deixar você confuso numa segunda olhada ao código.
  • Além do que foi apontado acima, uma sugestão para os próximos projetos é editar o readme com prints do projeto e informações relevantes, não podemos nos esquecer que ele, assim como o link do surge, também é parte do projeto e sempre é exigido pelas empresas nos processos seletivos. ;)

No mais, parabéns pela entrega, mandou bem!

Tenha uma ótima semana!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants