palantir/blueprint

TODO: Use react-focus-lock in overlay component

Open

#6676 aperta il 25 gen 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: coreType: enhancementhelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 24g 18h) (17 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): core v5.9+

Feature request

Follow-up to Overlay2 PR https://github.com/palantir/blueprint/pull/6656

We should try to use the react-focus-lock library to replace a bunch of bespoke focus management code dealing with focus traps and document event handlers.

Guida contributor

Direzione di ricerca
Rivedi la PR Overlay2 #6656 per comprendere il codice esistente di gestione del focus. Esamina il componente overlay attuale per identificare la logica personalizzata di trappola del focus che può essere sostituita. Quindi integra react focus lock avvolgendo il contenuto dell'overlay con il suo componente FocusLock, rimuovendo i gestori di eventi personalizzati. Assicurati che i test esistenti continuino a passare e che l'accessibilità sia preservata.
Tech stack
typescriptreact
Dominio
frontendaccessibility
Tipo issue
Funzionalità
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ReactTypeScriptFocus managementBlueprint Overlay2 PR #6656
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60