ndelvalle/v-click-outside

The isActive config is ignored when updated dynamically

Open

#143 aberto em 8 de nov. de 2019

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)JavaScript (959 stars) (90 forks)batch import
bughelp wanted

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

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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
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.
mostly clear
Prerequisites
Understanding of Vue reactivityFamiliarity with Vue directives
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35
Direção de pesquisa
The issue is that the `isActive` property in the config object is not being observed when changed dynamically. The fix likely involves adding a watcher or recomputing the directive's behavior when `isActive` changes. Check the source file `src/directive.js` for how the config is bound and if there's a mechanism to react to changes. The CodeSandbox linked in the issue (https://codesandbox.io/s/v click outside e4vin) demonstrates the problem. The assignee might have insights; review any existing comments or PRs for potential solutions.