PR de Correção

[ClaraMeirelles]

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações	Feito
O usuário consegue acessar a página Home, onde encontra uma lista com, no mínimo, 20 Pokémons	✅
Cada Pokémon da lista é representado como um card com opções de adicionar à Pokédex e ver detalhes	✅
O usuário consegue adicionar um Pokémon à sua Pokédex	✅
O usuário consegue remover um Pokémon da Pokédex	✅
Não é possível adicionar duas vezes o mesmo Pokémon na Pokédex	✅
Cabeçalho com botões para transitar entre Home e Pokédex	✅
O usuário consegue usar o botão no cabeçalho da página de Detalhes para adicionar ou remover Pokémon da Pokédex	-
O usuário consegue acessar a página da Pokédex, onde encontra uma lista dos Pokémons adicionados	✅
O usuário consegue abrir uma página de detalhes específicos de um Pokémon, esteja ele na Home ou na Pokédex	✅
Quão fácil foi achar as informações no site?	Médio
O que você achou do design do site?	Bom
[Desafio] Listas de Pokémons paginadas	✅
[Desafio] O usuário consegue fazer uma batalha entre dois Pokémons e há uma resposta de quem é o vitorioso	-
[Desafio] Todas as telas do site são responsivas	✅
Funcionalidade extra	-

Feedback do código	Feito
Requisições isoladas dos componentes	✅
Utilizou estado global para armazenar a lista de Pokémons	✅
Não 'forçou' a atualização com o método reload() ao remover um Pokémon	✅
Boa indentação	✅
Utilizou corretamente o router	✅
Todos os erros são tratados e exibidos adequadamente para o usuário	-
Criou link do surge	✅
Criou um bom readme com informações relevantes para o projeto	✅

Olá!

Primeiramente, parabéns pela entrega deste projeto!

O Layout ficou bem legal,mas tenho pequenos pitacos:

• A ideia de deixar o scroll dentro da página ficou muito legal, mas seria ainda melhor se a página não precisasse do scroll externo, exibindo todo o conteúdo dela e deixando o scroll apenas pra lista de pokémons 😁
• deixar mais claro pra que serve cada botão/ícone, de repente no hover colocar uma legenda, talvez... essa parte é importante também em questão de acessibilidade (colocar o que cada ícone faz na propriedade alt do elemento já ajuda bastante!)
• A fonte Times New Roman (padrão) é um tanto formal demais pra esse site, talvez fosse mais adequado uma fonte menos "séria" para uma página de pokémons 😅
• Mudar o ícone do cursor quando for um texto clicável (mudança de páginas), igual acontece nos ícones de detalhes e adicionar ajuda bastante a navegabilidade 😬
• Na tela detalhes aparecem dois setores, cada um com uma foto e um tipo do pokémon... parece que o bulbassauro de frente é tipo grama e de costas é tipo veneno 😅 acho que essas informações poderiam estar junto com os outros dados do pokémon 😬

Sobre a navegabilidade, achei um pouco confuso a troca de páginas, onde a logo do site é o que define a rota de voltar para a tela inicial na tela de detalhes, por exemplo, já que condicionalmente o usuário busca um botão à esquerda da tela para voltar à tela anterior, talvez adicionar um botão "voltar" seria uma boa adição pra navegabilidade.

Um fato estranho da tela detalhes é: eu só consigo acessar os detalhes dos 6 primeiros pokémons de cada página, o mesmo acontece na pokédex 🤔

Sobre a organização do código, a melhor prática seria fazer a requisição em um hook, separado do Estado Global, para que cada arquivo tivesse apenas uma funcionalidade, sem precisar se responsabilizar por mais de uma coisa ao mesmo tempo, assim o debug fica mais fácil também 😉

Faltou criar um .catch para as requisições no GlobalState, se houver algum erro da API, o usuário não terá nenhum retorno 😐, é legal sempre lembrar que pode haver erros, principalmente em APIs públicas, então vale a lembrança de cuidar para que todos os erros sejam tratados.

Vi que vocês criaram um readme, mas para ele estar mais completo ainda, é legal trazer quais as tecnologias usadas no projeto (React, axios, styled-components, etc), além do nome de quem participou do projeto. Uma seção de como usar o site é uma boa adição ao Readme também 😉

No mais, parabéns pela entrega!

Tenham uma excelente semana!