-
Notifications
You must be signed in to change notification settings - Fork 5
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
Comments
@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. |
@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). |
- Change variable ngo_id to camel case Refs boacausa#114
Novo layout de filtros será implementado na issue: #143 |
|
@anibiamachado sobre o ultimo comentário, muito bem colocado, adicionei uma issue pra isso: #146 |
Oi @anibiamachado , @carolinesalib o que ficou pendente nessa task foi só o item 3
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. Os demais items foram ou estão sendo verificados em outras tasks. :) |
Então, está tudo ok @kellynvd e @carolinesalib ;) |
Planejar como vai ser os filtros e implementar (ou criar issues diferentes para implementar).
Issue origem: #40
The text was updated successfully, but these errors were encountered: