Preparando revisão do código #1
Conversation
|
Someone is attempting to deploy a commit to a Personal Account owned by @Mateusr337 on Vercel. @Mateusr337 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! ;) 📓
🚀
| export const AuthProvider = (props) => { | ||
| const [user, setUser] = useState(null); | ||
| const [progress, setProgress] = useState(null); | ||
| const [navbar, setNavbar] = useState(false); |
There was a problem hiding this comment.
é onde tem a opção de deslogar do app, para limpar o localStorange
| const Conatiner = styled.div` | ||
| width: 100%; | ||
| height: 60px; | ||
| padding: 20px; | ||
|
|
||
| background-color: #EA5766; | ||
| ${props => props.finished && 'background: #8CC654;'} | ||
| border-radius: 10px; | ||
|
|
||
| display: flex; | ||
| justify-content: space-between; | ||
| align-items: center; | ||
| ` |
| @@ -1,13 +0,0 @@ | |||
| import styled from "styled-components" | |||
|
|
|||
| export default function Menssage({ text }) { | |||
There was a problem hiding this comment.
Não é uma boa prática colocar hifén no nome dos componentes ou pastas.
O recomendado é seguir o padrão camelCase
| const Container = styled.div` | ||
| height: 100vh; | ||
|
|
||
| align-items: center; | ||
| justify-content: center; | ||
| gap: 20px; | ||
|
|
||
| display: flex; | ||
| flex-direction: column; | ||
|
|
||
| form { | ||
| width: 100%; | ||
|
|
||
| align-items: center; | ||
| gap: 6px; | ||
|
|
||
| display: flex; | ||
| flex-direction: column; | ||
|
|
||
| button { |
There was a problem hiding this comment.
Você poderia ter criado componentes para os inputs ou até o foms. Tendo em vista que os mesmos não diferem muito da tela de login
| @@ -1,22 +0,0 @@ | |||
| @import url('https://fonts.googleapis.com/css2?family=Lexend+Deca:wght@100;200;300;400;500;600;700;800;900&family=Roboto:ital,wght@0,100;0,400;0,500;0,700;0,900;1,100;1,300;1,400;1,500;1,700;1,900&display=swap'); | |||
There was a problem hiding this comment.
Você poderia ter importado a font com a tag link no index.html
3 participants
No description provided.