Skip to content

Conversation

@gabrielmacieldev
Copy link
Owner

No description provided.

@gabrielmacieldev gabrielmacieldev merged commit eb4bd91 into main Jan 7, 2023
@ClaraMeirelles
Copy link

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Tem 3 páginas: Home, Pokedex e Detalhes;
Projeto segue o design do figma
O fluxo de trocas de páginas segue igual ao fluxograma
Página Home tem uma lista de Pokemons, contendo ao menos 20 Pokemons
Página Home tem cada Pokemon representado por um Card
Em cada card de Pokemon tem um botão para adicioná-lo à Pokedex e um outro botão para acessar os detalhes do Pokemon
O Header página Home tem um botão para acessar a página da Pokedex
A Pokédex renderiza a lista de pokémons adicionados na pokedex
Na Pokédex tem em cada card de Pokemon um botão para removê-lo da Pokedex e um outro botão para acessar os detalhes do Pokemon
Não deve ser possível adicionar o mesmo Pokemon duas vezes na Pokedex
A página de detalhes mostra os detalhes do Pokemon selecionado de forma coerente com o design
O Header da página de detalhes deve ter um botão para adicionar ou remover da Pokedex e outro para voltar a página home. -
Usou o React Router
Usou o React Context e declarou um Estado Global
Fez requisições com o Axios
O que você achou do design do site? Podemos melhorar
Criou link do surge
Criou um bom readme com informações relevantes para o projeto -
Feedback do código Feito

image

Olá Gabriel!

Sobre o layout, você acabou deixando as medidas a absolutas em alguns elementos, o que faz com que o seu site "vaze" da tela, a depender do tamanho desta. O ideal é usar as medidas relativas, para que se encaixe melhor.
Com isso, o botão do header acabou nunca ficando aparente na tela, apenas com a rolagem para o lado, o que dificulta a usabilidade do site.
Há também alguns outros detalhes do layout que poderiam ser melhorados, mas no geral está muito bom!

Parabéns pela entrega e por sua dedicação!

Bons estudos!

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.

3 participants