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

Funcionalidade do botão de pesquisa #73

Open
pedroespindula opened this issue Mar 13, 2019 · 4 comments · Fixed by #114
Open

Funcionalidade do botão de pesquisa #73

pedroespindula opened this issue Mar 13, 2019 · 4 comments · Fixed by #114
Labels
Aprimoramento Nova feature ou solicitação de melhora Bug Algo não está funcionando como deveria Feature Nova funcionalidade Frontend Relacionado à Frontend

Comments

@pedroespindula
Copy link
Contributor

Descrição da melhoria:
Botão de pesquisa poderia ser usado para procurar em todos os termos.

Necessidade da melhoria:
Atualmente o botão de pesquisa não apresenta funcionalidade.

Como ela pode ser implementada:
Criação de um filter na lista de acronimos com o resultado do input.

@pedroespindula pedroespindula added Aprimoramento Nova feature ou solicitação de melhora Frontend Relacionado à Frontend labels Mar 13, 2019
@JoseRenan JoseRenan added the Bug Algo não está funcionando como deveria label Mar 17, 2019
@pedroespindula pedroespindula self-assigned this Mar 17, 2019
@pedroespindula pedroespindula added the Feature Nova funcionalidade label Apr 7, 2019
@fanny
Copy link
Contributor

fanny commented May 26, 2019

Eu tava debugando, e vi que o componente que tem o botão é o search, nele vocês usam uma biblioteca externa pra definir o select e passam esse handler pra ele.

O problema é que, como esse input não é gerenciado por vocês, é necessário passar pra os listeners da biblioteca externa, só que não tem como pegar o evento quando o cara terminou de digitar(posso tá errada, olhei nesse link aqui)

Alternativas que eu pensei:

  1. Por quê não fazer nosso próprio select? (e a que eu mais gosto)
  2. O método que a gnt poderia usar até segunda via, que funcionaria, seria o onBlur, porque pra pessoa clicar no botão ela teria que tirar o foco do select(acho meio gambiaroso, mas poderia ser algo que ficasse valendo pra uma possível segunda versão com uma mudança grande)
  3. Por enquanto, deixar só o select(até decidir como fica tudo)

@fanny
Copy link
Contributor

fanny commented May 26, 2019

Aproveitando o ensejo, por quê o Glossary não é um componente funcional?

Sugestão:
Eu vi que aqui vocês usam renderização condicional, acho que dá pra substituir esse trecho

handleSelect={this.handleAcronymChange}/>
{this.isSearchEmpty() ?
<DayPhrase entry={randomEntry}/> :
""
}

por:

      {this.isSearchEmpty() && <DayPhrase entry={randomEntry}/>}

Pra não precisar passar uma string vazia, o mesmo vale pra o que tem abaixo

@thayannevls
Copy link
Contributor

@pedroespindula

@pedroespindula pedroespindula removed their assignment Jul 23, 2019
@fanny fanny self-assigned this Jul 30, 2019
@fanny fanny mentioned this issue Aug 2, 2019
@pedroespindula pedroespindula reopened this Oct 2, 2019
@pedroespindula
Copy link
Contributor Author

Essa issue foi fechada automaticamente mas ainda não foi resolvida por isso foi reaberta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aprimoramento Nova feature ou solicitação de melhora Bug Algo não está funcionando como deveria Feature Nova funcionalidade Frontend Relacionado à Frontend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants