Conversation
|
Someone is attempting to deploy a commit to a Personal Account owned by @mdeosjr on Vercel. @mdeosjr first needs to authorize it. |
leleoRA
left a comment
leleoRA
left a comment
There was a problem hiding this comment.
Falaaaa artista da programação!! 🎨 Tudo na paz??
Seguinte... Primeiramente beba sempre bastante água 💧 srsrsrsr (saúde em primeiro lugar) ⚕️ .
Em relação ao projeto, parabéns!
Fico muito contente em ver que você persistiu e conseguiu terminar o projeto.
Isso é muito importante para nós da Driven 🥳
Obs:Como geralmente não temos feedback de código nos projetões, nesse projeto vamos focar nos seguintes critérios: Styled-componentes, ContextApi e o uso das bibliotecas solicitadas. Então por esse motivos alguns pontos não serão levados em consideração.
Deixei alguns comentários no seu projeto. 💬
Aguardo o seu feedback! ;) 📓
🚀
| <ContextoUsuario.Provider value={{imagemUsuario, setImagemUsuarioLocal}}> | ||
| <BrowserRouter> | ||
| <ContextoToken.Provider value={{token, setTokenLocal}}> |
There was a problem hiding this comment.
Seria interessante ter criado um contexto unico; Tendo em vista que ambas as informações são referentes ao usuário
| const Titulo = styled.h1` | ||
| font: normal 22.976px 'Lexend Deca'; | ||
| line-height: 29px; | ||
|
|
||
| color: #126BA5; | ||
|
|
||
| padding-top: 98px; | ||
| `; | ||
|
|
||
| const Subtitulo = styled.h2` | ||
| font: normal 17.796px 'Lexend Deca'; | ||
| line-height: 22px; | ||
|
|
||
| color: ${props => props.porcentagem === 0 ? '#BABABA' : '#8FC549'} | ||
| ` |
There was a problem hiding this comment.
Pelo nome do arquivo, fica implícito que ele só vai retornar o componente de titulo.
Dessa forma que você está fazendo pode confundir até você mesmo na hora de uma manutenção.
O recomendado seria criar outro arquivo para o componente subtitulo
2 participants
No description provided.