Adicionar filtros na página dos pets para adoção

boacausa / webplatform

Por uma boa causa - Para fazer do mundo um lugar melhor para todos

https://www.boacausa.org

6 stars 5 forks source link

Adicionar filtros na página dos pets para adoção #114

Open carolinesalib opened 4 years ago

carolinesalib commented 4 years ago

Planejar como vai ser os filtros e implementar (ou criar issues diferentes para implementar).

Filtro de cidade?
Filtro de ONG?
Filtro por palavra chave?
Filtro por sexo?

Issue origem: https://github.com/boacausa/webplatform/issues/40

kellynvd commented 4 years ago

@carolinesalib PR disponível para review. :)

Eu fiquei em dúvida sobre o layout então por enquanto só mantive o que você tinha iniciado, só ajustei o espaçamento. Mas posso para criar uma issue separada se for para mudar, e também para ajustar os filtros na página quando for mobile.

Screenshot from 2020-01-23 10 50 01

carolinesalib commented 4 years ago

@kellynvd PR aprovado. 🎉

Realmente o layout era o outro, esse é o layout que eu desenhei no comecinho, mas depois o Hedie desenhou um melhor. Separei numa pasta chamada "Main styles" do Figma. Pode criar uma nova issue pra ajustar? Dai ja aproveita e coloca na issue pra pensar a respeito de responsividade nos filtros.

Ps: eu mergi o menu lateral antes de fazer o PR e agora ta com um conflito, acho que é só identação, depois que corrigir ja pode mergir (o deploy roda automatico).

carolinesalib commented 4 years ago

Novo layout de filtros será implementado na issue: https://github.com/boacausa/webplatform/issues/143

anibiamachado commented 4 years ago

Sobre o layout: vi que existe outra issue para ajustar, mas para deixar registrado, acho legal seguir a linha da barra de menus, mais arredondado;
Creio que irão ajustar para versão mobile ainda, certo? Pois percebi que lá os campos ficaram pequenos, impossibilitando de ler o que está escrito dentro;
Quando tento buscar por palavra-chave e informo, por exemplo, "to", está trazendo os pets "Carson" e "Freeda";
Uma das ONGs está sem a informação da cidade no cadastro, essa informação será obrigatória no cadastro da ONG? Pois, nesse caso, como está vazio, aparece no filtro um registro vazio para filtrar e quando tento utilizar ele, não funciona.

carolinesalib commented 4 years ago

@anibiamachado sobre o ultimo comentário, muito bem colocado, adicionei uma issue pra isso: https://github.com/boacausa/webplatform/issues/146

kellynvd commented 4 years ago

Oi @anibiamachado , @carolinesalib o que ficou pendente nessa task foi só o item 3

Quando tento buscar por palavra-chave e informo, por exemplo, "to", está trazendo os pets "Carson" e "Freeda";

Porém ele está funcionando corretamento pois o filtro busca por nome e descrição, então "Carson" e "Freeda" aparecem por terem "to" em alguma palavra na descrição.

Screenshot from 2020-03-20 16 21 18

Os demais items foram ou estão sendo verificados em outras tasks. :)

anibiamachado commented 4 years ago

Então, está tudo ok @kellynvd e @carolinesalib ;)