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

Add the filters form to a collapsible container #154

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

rogersouza
Copy link

PR parcial da #138

Aqui estão as alterações referentes a primeira parte da issue. Não entendi muito bem a segunda, se você puder explicar melhor fico feliz em adicionar a mudança a esse PR

😃

@turicas
Copy link
Owner

turicas commented Feb 7, 2019

Oi, Roger, obrigado pela contribuição! Vamos a alguns detalhes:

  • Sobre os filtros, o ideal seria deixar os filtros sempre escondidos caso o usuário não esteja filtrando nada (ou seja, caso a query string esteja vazia) - a ideia inicial era justamente diminuir a altura dessa tela (então quem quiser filtrar terá que clicar uma vez para expandir o formulário).
  • A segunda parte é basicamente adicionar uma frase em destaque, provavelmente no topo da tabela ou da página; alterei a descrição da issue pra ficar mais explícita.

Você consegue implementar essas alterações e subir os commits para esse branch do PR, por favor?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants