gajus/react-outside-event

Error on missing onOutsideEvent

Open

#3 aperta il 3 mag 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (9 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (44 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

If I wrap an element that does not define an onOutsideEvent method, the following error is thrown

Component does not define "onOutsideEvent" method.

I can fix this by defining an empty method for onOutsideEvent. Would it be better to remove this error or issue a warning instead of an error?

Guida contributor

Direzione di ricerca
Ispeziona il codice sorgente dell'HOC per capire come viene chiamato onOutsideEvent e considera di trasformare l'errore in un avviso o rimuoverlo.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80