ndelvalle/v-click-outside

The isActive config is ignored when updated dynamically

Aberta

#143 aberto em 8 de nov. de 2019

Ver no GitHub
 (2 comentários) (0 reação) (1 responsável)JavaScript (90 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (959 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

I'm trying to disable the behavior of the click outside dynamically by changing the isActive property of the config object but the behavior still the same.

I know I can use the middleware to handle this (check line 36) but I don't know if the config of click outside expects to be dynamically.

https://codesandbox.io/s/v-click-outside-e4vin

Guia do colaborador

Direção de pesquisa
O problema é que a propriedade `isActive` no objeto de configuração não está sendo observada quando alterada dinamicamente. A correção provavelmente envolve adicionar um observador ou recalcular o comportamento da diretiva quando `isActive` muda. Verifique o arquivo de origem `src/directive.js` para ver como a configuração é vinculada e se há um mecanismo para reagir a alterações. O CodeSandbox vinculado na issue (https://codesandbox.io/s/v click outside e4vin) demonstra o problema. O responsável pode ter insights; revise quaisquer comentários ou PRs existentes para possíveis soluções.
Pilha de tecnologia
javascriptvue
Domain
frontend
Tipo Issue
Bug
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Vue
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75