Conversation
header e footer
Merge pull request #1 from future4code/master
There was a problem hiding this comment.
Oi gente! O projeto de vocês foi avaliado como abaixo do esperado. Isso porque das três funcionalidades requisitadas (produtos, filtros e carrinho), vocês só conseguiram implementar uma.
Mas o que vocês fizeram ficou bem legal, o site está bonito apesar de ter alguns problemas de tamanhos (o carrossel não ocupa a tela inteira dependendo do tamanho da tela, o footer para mim aparece bem fininho e sem nada dentro, etc... alguns detalhezinhos).
Então eu acho que talvez o que faltou por aí foi mais uma questão de organização mesmo, e que com mais tempo teria dado pra fazer algo bem completo!
Para os próximos projetos, eu diria para focar mais em fazer tudo funcionar da maneira mais boba possível e depois começar a deixar o site mais bonitinho. Não adianta muito ter um site maravilhoso que não funciona né?
E daí eu sei que se organizar em grupo é mais difícil mas com prática vocês vão notando como cada um funciona melhor, o curso é uma oportunidade de praticar essas habilidades em um ambiente seguro.
Outras coisas que vocês poderiam fazer para deixar o projeto ainda melhor:
- Editem o arquivo Readme! Coloquem os inegrantes do grupo,o que funciona ou não e, principalmente, o link do surge do projeto de vocês.
- Padronizem a estilização do projeto: usem apenas CSS puro ou styled-components (mais recomendado)
Se ficou alguma dúvida de como fazer algo no projeto, vocês podem assistir aos vídeos do Thinkin in React e, em breve, soltarei um documento com os erros mais comuns e suas soluções.
É isso, agora vamos focar nos próximos projetos que ainda tem bastante coisa pela frente! 😄
| <h3>iPad Pro 11” Apple Wi-Fi + Cellular 64GB - Cinza Espacial</h3> | ||
| <p>de R$ 8.999,00</p> | ||
| <h4>por R$ 7.524,91</h4> | ||
| <a href="" class="botao botaoCarrinho">Colocar no carrinho</a> |
There was a problem hiding this comment.
Atenção! No React usamos className, não class
3 participants
PR de Correção