ndelvalle/v-click-outside

The isActive config is ignored when updated dynamically

Ouverte

#143 ouverte le 8 nov. 2019

Voir sur GitHub
 (2 commentaires) (0 réaction) (1 personne assignée)JavaScript (90 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (959 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Guide contributeur

Direction de recherche
Le problème est que la propriété `isActive` dans l'objet de configuration n'est pas observée lorsqu'elle est modifiée dynamiquement. La correction implique probablement d'ajouter un watcher ou de recalculer le comportement de la directive lorsque `isActive` change. Vérifiez le fichier source `src/directive.js` pour voir comment la configuration est liée et s'il existe un mécanisme pour réagir aux changements. Le CodeSandbox lié dans l'issue (https://codesandbox.io/s/v click outside e4vin) démontre le problème. L'assigné pourrait avoir des idées ; examinez les commentaires ou PR existants pour des solutions potentielles.
Stack technique
javascriptvue
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Vue
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75