Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
carolinesalib opened this issue Nov 22, 2019 · 7 comments
Open

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

carolinesalib opened this issue Nov 22, 2019 · 7 comments
Assignees
Labels

Comments

@carolinesalib
Copy link
Member

carolinesalib commented Nov 22, 2019

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: #40

@carolinesalib carolinesalib created this issue from a note in Boa Causa Dev (To do) Nov 22, 2019
@kellynvd kellynvd moved this from To do to In progress in Boa Causa Dev Jan 16, 2020
@kellynvd kellynvd self-assigned this Jan 16, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 17, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 21, 2020
@kellynvd kellynvd moved this from In progress to In review in Boa Causa Dev Jan 23, 2020
@kellynvd
Copy link
Member

@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
Copy link
Member Author

@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).

kellynvd added a commit to kellynvd/webplatform that referenced this issue Jan 23, 2020
-  Change variable ngo_id to camel case

Refs boacausa#114
carolinesalib added a commit to kellynvd/webplatform that referenced this issue Jan 24, 2020
@carolinesalib carolinesalib moved this from In review to Ready for test in Boa Causa Dev Jan 28, 2020
@carolinesalib
Copy link
Member Author

Novo layout de filtros será implementado na issue: #143

@anibiamachado
Copy link

  • 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
Copy link
Member Author

@anibiamachado sobre o ultimo comentário, muito bem colocado, adicionei uma issue pra isso: #146

@kellynvd
Copy link
Member

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. :)

@kellynvd kellynvd moved this from In progress to Ready for test in Boa Causa Dev Mar 20, 2020
@anibiamachado
Copy link

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

@anibiamachado anibiamachado moved this from Ready for test to Done in Boa Causa Dev Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Boa Causa Dev
  
Done
Development

No branches or pull requests

3 participants