palantir/blueprint

[Select] Can't select item by keyboard - Enter

Open

#5191 aperta il 22 mar 2022

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

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

Can't select item by keyboard Enter If replace Button via native button html tag, and set filterable={false}.

Environment

  • Package version(s):
  • Operating System:
  • Browser name and version:

Code Sandbox

Link to a minimal repro: https://codesandbox.io/s/blueprint-sandbox-forked-x7z0uz?file=/src/SelectExample.tsx

Loom: https://www.loom.com/share/2639f8abedff41ac84fe02a7548f0574

Steps to reproduce

  1. child components - native button
  2. filterable={false}
  3. Use Keyboard to select active item

Actual behavior

Can't select item, only if i pressed Enter second time

Expected behavior

Select item by keyboard Enter

Guida contributor

Direzione di ricerca
Indaga sulla gestione degli eventi da tastiera del componente Select quando `filterable={false}` e l'elemento è un `<button>` nativo. Controlla la sandbox del codice per il comportamento esatto. Cerca nel codice sorgente della repository il componente Select, in particolare i gestori `onKeyDown` o simili che gestiscono la selezione con il tasto Invio. Il problema probabilmente coinvolge una condizione che salta la selezione alla prima pressione a causa di filterable=false. Esamina i commenti correlati nell'issue per ulteriori indizi.
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
ReactGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60