Open
Conversation
Postman, routers and screens
Component CardPokemon
Style page details
Router, Pagina de Erro, e Estilização Error
Home/card Estilizados, Espaçamento no Headear
Style atual details
Utilização do global state, renderização dos pokemons
Poke allison
Botões de acordo com a Pagina
Estilização do Button
Function do goToPage por props antigo
Renderização dos pokemons escolhidos na pokedex
Adicionada a função de remover da pokedex
Botões de Add e Remove
Animação de loading
Funcionalidades
Primeira revisão no código
Ajuste da fonte para pegar em todo o site
… Button do Menu | Organizando Componente do Button | e entre outro :)
Poke allison
Details button
leticia-chijo
commented
Feb 23, 2021
Author
leticia-chijo
left a comment
leticia-chijo
left a comment
There was a problem hiding this comment.
Olá! O projeto de vocês foi considerado acima do esperado, parabéns! 🎉
O layout está bonito e o site funciona do jeito que foi pedido, além de ser responsivo. Algumas observações:
- Na página da pokédex, quando eu ainda não adicionei nenhum pokémon, ele fica carregando infinitamente. Acho que nessa tela, teoricamente, nem precisa colocar loading! Esses dados são locais, não tem nenhuma requisição para pegar eles, então ele não deveria demorar para aparecer. Um placeholder de "Você ainda não tem nenhum pokémon, adicione!" ou algo assim seria mais apropriado nesse caso, só pra não deixar vazio mesmo.
- Vocês deixaram um PR aberto! O conteúdo dele não foi considerado para a correção, apenas o que está na master. Lembrem de fechar tudo nos próximos 😄
- O código está bem organizado e gostei de como vocês guardaram as informações
Acho que é isso! Se algum ponto ainda não ficou claro, assistam ao vídeo com a sugestão de resolução que enviamos na sexta e, se sobrar alguma dúvida, por favor mandem lá no canal! ❤️
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR de Correção