ndelvalle/v-click-outside

The isActive config is ignored when updated dynamically

Offen

#143 geöffnet am 08.11.2019

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (90 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (959 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem ist, dass die Eigenschaft `isActive` im Konfigurationsobjekt nicht beobachtet wird, wenn sie dynamisch geändert wird. Die Lösung beinhaltet wahrscheinlich das Hinzufügen eines Watchers oder die Neuberechnung des Verhaltens der Direktive, wenn sich `isActive` ändert. Überprüfen Sie die Quelldatei `src/directive.js`, um zu sehen, wie die Konfiguration gebunden ist und ob es einen Mechanismus gibt, um auf Änderungen zu reagieren. Der in der Issue verlinkte CodeSandbox (https://codesandbox.io/s/v click outside e4vin) demonstriert das Problem. Der Zugewiesene könnte Einblicke haben; überprüfen Sie vorhandene Kommentare oder PRs auf mögliche Lösungen.
Tech Stack
javascriptvue
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Vue
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75