Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jan 14, 2021

Oie! Esse é um PR de correção que nao precisa ser mergeado. Os comentarios estarao um pouco mais abaixo na pagina!

@@ -1 +1,4 @@
Sejam bem vindos ao Labe-commerce, esse repositório contém um esqueleto de app React e um .gitignore.
Trata-se de um trabalho desenvolvido em dupla pelo Wilson Ferreira Junior e Pablo Silas como projeto semanal do curso intensivo da Labenu.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabens pelo cuidado com o README, muito bom!! Lembrem-se da proxima vez de colocar o link do surge! Inclusive, a aplicacao ficou bem bonita!

padding-left: 10px;
`
// Define o rodapé
const Footer = styled.footer `
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O footer ficou bonito, mas nao funciona muito bem. O lugar onde ele fica atualmente depende do container de produtos. Seria legal fixar ele no fim da pagina de alguma forma!

Comment on lines +5 to +17
const BoxFilter = styled.div`
width: 80%;
height: 14vh;
display: flex;
justify-content: center;
background-color: #202124;
box-sizing:border-box;
padding: 10px;
border-radius: 0px 0px 10px 10px;
margin-top: 0px;
margin-left: 10px;
`
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voces fizeram um bom trabalho de estilizacao, mas faltou um pouco de atenção com a identacao! Fiquem de olho nos proximos projetos

@ghost
Copy link
Author

ghost commented Jan 14, 2021

Oi grupo! A entrega de voces está acima do esperado, parabéns!

O código está bem escrito, as funcionalidades obrigatorias estão ok e voces ainda se preocuparam muito com a estilizacao!

Continuem assim!

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.

2 participants