corejavascript/typeahead.js

autocomplete() should trigger autocomplete event even when text already matches

Open

#132 aberto em 15 de mar. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (956 stars) (232 forks)batch import
enhancementhelp wanted

Description

Currently, autocomplete() doesn't trigger the autocomplete event if the input text matches the given $selectable. Why should it, if autocompletion won't change the text?

Because there may be other data attached to the typeahead choice which is extracted by downstream logic listening for the event.

Instead of

      isValid = data && query !== data.val;
      if (isValid && !this.eventBus.before('autocomplete', data.obj, data.dataset)) {

it should be just

      if (data && !this.eventBus.before('autocomplete', data.obj, data.dataset)) {

Guia do colaborador

Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptEvent systemTypeahead library
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
Review the autocomplete() method in src/typeahead/typeahead.js around line 371. Understand the current condition that prevents event dispatch when query matches. Implement the suggested change to remove the query !== data.val check. Ensure that the autocomplete event is emitted even when text matches, so that downstream listeners can access the data object.