Semantic-Org/Semantic-UI-React

Rework TransitionablePortal callbacks for consistency

Open

#2423 aperta il 10 gen 2018

Vedi su GitHub
 (9 commenti) (0 reazioni) (0 assegnatari)JavaScript (4065 fork)batch import
bughelp wanted

Metriche repository

Star
 (13.136 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Steps

  1. Create a controlled <TransitionablePortal />
  2. Toggle the open prop from false to true
  3. Trigger a close with ESC or click outside

Expected Result

  1. On prop change to open, the onOpen callback should not be called. This is a programmatic prop change, not a trigger click.
  2. When a close event is triggered, the onClose callback should be called before starting the transition as the user may choose to disregard the close request.

Actual Result

  1. On prop change to open, onOpen is called at the beginning and end of the open transition
  2. When a close event is triggered, the onClose callback is called after the onHide callback. There is no way to handle the close request and ignore it.

See the test case for proposed fixes.

Version

0.77.2

Testcase

https://codesandbox.io/s/rj8qv1yl2o

Guida contributor

Direzione di ricerca
Esamina l'implementazione del callback del componente TransitionablePortal nel codice sorgente di Semantic UI React. Concentrati sulla tempistica delle chiamate onOpen e onClose rispetto alle transizioni e alle modifiche delle prop. Crea un test case per verificare il comportamento atteso e regola di conseguenza la logica del callback.
Tech stack
reactjavascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
ReactSemantic UI React
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45