adobe/react-spectrum

onCancel should fire in AlertDialog if Escape is pressed

Open

#1773 aperta il 7 apr 2021

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)TypeScript (1500 fork)auto 404
good first issue

Metriche repository

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

Descrizione

🐛 Bug Report (or maybe feature request?)

When adding an onCancel handler to an AlertDialog, it should fire if the dialog is closed via the Escape key.

See here for a repro: https://codesandbox.io/s/adoring-meadow-7cl4c?file=/src/App.js

Open the dialog, click Cancel, dialog will close, you'll see 'Canceled' in the console. Open the dialog, press Escape key, dialog will close, no 'Canceled' in the console.

Guida contributor

Direzione di ricerca
Ispeziona il componente AlertDialog per vedere come viene gestito il tasto Escape. Verifica se esiste un listener per l'evento keydown e se attiva il callback onCancel. Cerca eventuali logiche di propagazione o condizionali mancanti che impediscono a onCancel di essere attivato quando viene premuto Escape.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
React
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70