Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

Acumula descrição dos filtros ao invés de exibir apenas a do último aplicado #39

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

giovanisleite
Copy link

Closes #35

Para acumular as descrições separei as descrições dos filtros em dois grupos: as descrições sobre a pessoa (genero e etnia) e a descrição da carreira política.

Genero e etnia são cumulativos, optei por usar sempre a mesma ordem, primeiro genero e depois etnia, na frase (não importando a ordem do filtro). Então, aplicando os filtros Quantas são mulheres? e depois Quantos são negros ou pardos? (Exemplo 1) exibe a mesma descrição que aplicando Quantos são negros ou pardos? e depois Quantas são mulheres?.

Já os filtros relacionados a carreira política, apesar de nunca concorreram ser um subgrupo de nunca se elegeram, não acumulei a descrição desses porque acho que se aplicarem os dois, é melhor descrever apenas com o grupo menor (Exemplo 2).

Para juntar a descrição da pessoa com a descrição da carreira política coloquei um "que" entre eles (Exemplo 3).

Exemplo 1:

Aplicando as perguntas:

  • Quantas são mulheres?
  • Quantos são negros ou pardos?

O resultado é: Destas, 157 são mulheres de cor negra, parda ou indígena


Exemplo 2:

Aplicando as perguntas:

  • Quantos nunca foram eleitos?
  • Quantos nunca concorreram?

O resultado é Destes, 726 nunca concorreram numa eleição


Exemplo 3:

Aplicando as perguntas:

  • Quantos nunca concorreram?
  • Quantas são mulheres?

O resultado é Destes, 281 são mulheres que nunca se elegeram

… inves de apenas o ultimo

Separei as funçoes de descriçao de filtros em filtros relacionados a pessoa (genero e etnia) e filtros relacionados a carreira politica (ja foi eleito, ja se candidatou, nunca se candidatou). Estando presentes, a descriçao dos filtros sera exibida na ordem: genero, etnia, carreira politica
@sergiomario sergiomario added the hacktoberfest-accepted Hacktoberfest label label Oct 28, 2020
@lguima
Copy link

lguima commented Oct 9, 2022

@giuliocc acho que podemos encerrar essa PR, pois a estrutura do projeto mudou, esse arquivo não existe mais.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hacktoberfest-accepted Hacktoberfest label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Melhorias nos filtros
3 participants