Conversation
layout basico do component AddProduto(vendedor) finalizado
logica do componente PostProduto implementada
Atualizado o input forma de pag. para select
layout card produto finalizado
Layout Item Carrinho
layout home filtro finalizado
navbar feito. Ainda faltam ajustes
carouselV1
carousel adicionado na home
Filtro nav bar funcionando com HomeFiltro
Lógica do Item do Carrinho
filtros max e min e ordenacao aplicado
Correções no carrinho
…zon e a funcao para isso esta no Component PostProduto
leticia-chijo
left a comment
leticia-chijo
left a comment
There was a problem hiding this comment.
Oi gente! Deixei uns comentários de melhorias no meio do código mas aqui vou falar de uma maneira mais geral.
Primeiramente: parabéns! Acho que todas as funcionalidades estão fazendo o que deveriam e isso é o mais importante do projeto. As sugestões que eu deu no código e que eu vou colocar aqui seria só pra deixar um site que já está muito bom ainda melhor!
A primeira coisa seria sobre a barra de busca. Ela só faz sentido na tela de produtos né? Então nas outras telas, eu esconderia essa barra.
A inspiração para o site de vocês era o enjoei, então achei que vocês poderiam ter aproveitado mais ele como referência visual! Dá até pra olhar o código css do site usando o inspetor 😉
Já em questão de UX, também achei o site legal e só teve uma coisinha que me incomodou: quando o site tá fazendo alguma requisição que demora, o usuário fica sem feedback e parece que nada tá acontecendo. Por exemplo, quando enviamos um novo produto pelo form. Ele demora um pouquinho até dar sucesso, o ideal seria ter algum tipo de indicação visual de que o pedido está sendo processado (como um loading).
Usar o material era um dos requisitos do projeto e vocês conseguiram fazer isso muito bem!
Mas basicamente é isso gente, nem tenho muito pra dizer porque o projeto ficou bem legalzão, acho que o visual de maneira geral poderia melhorar mas isso é um comentário meio arbitrário né? Haha. Beleza é um troço relativo mesmo 😝
Parabéns! 🎉
|
|
||
|
|
||
| render() { | ||
| console.log(this.state) |
There was a problem hiding this comment.
Antes de entregar os projetos, é legal apagar todos os console.logs de testes que vocês deixaram por aí! Pra facilitar, é só fazer uma busca no projeto por console.log e ir apagando 😄
| alert('Compra Finalizada! Volte Sempre!') | ||
| } | ||
|
|
||
| addQuantity = (id) => { |
There was a problem hiding this comment.
O carrinho de vocês tá funcionando direitinho, e essa era uma das partes mais desafiadoras do projeto mesmo... Parabéns!
|
|
||
| render() { | ||
|
|
||
| const novaListaCarrinho = this.state.itemsCarrinho.filter(item => { |
There was a problem hiding this comment.
Acho legal escrever o nome das coisas ou tudo em português ou tudo em inglês no projeto inteiro, só pra não virar bagunça haha
| return ( | ||
| <MainDiv> | ||
|
|
||
| <Carousel { ...settings }> |
There was a problem hiding this comment.
O carrossel ficou muuuito legal, adorei!!
| <TextField value={props.maximo} onChange={props.valorMax}id="outlined-search" label="max." type="number" variant="outlined" /> | ||
| </form> | ||
| <DivSeletor> | ||
| <FormControl component="fieldset"> |
There was a problem hiding this comment.
O filtro é outra parte um pouco mais complexa do projeto e o de vocês está funcionando direitinho! Minha única crítica é que, num primeiro momento, eu não entendi que eu tinha que clicar para os filtros aparecerem e que um dos campos, o nome, tá sem margem!
| <CustomSlider/> | ||
| <TabFiltros> | ||
| <ButtonFiltro> | ||
| <LinkButton color='#4D4D59' onClick={this.filtroDefault} href="#"> |
There was a problem hiding this comment.
Wooow, nem tinha notado no site que cada letra era de uma cor, que fofo! Haha
|
|
||
| <div className={classes.buttons}> | ||
|
|
||
| <Button variant="outlined" > |
There was a problem hiding this comment.
Já que esse botão não tá levando pra lugar nenhum, eu tiraria ele!
| <Form variant="outlined" > | ||
| <InputLabel htmlFor="outlined-age-native-simple">Forma de Pag.</InputLabel> | ||
| <Select | ||
| native |
There was a problem hiding this comment.
Gostei que vocês usaram select pra categoria e pagamentos, faz bastante sentido para o usuário final.
5 participants
PR de Correção