palantir/blueprint

Blueprint table scrolling problem when using keyboard

Open

#3835 aperta il 11 nov 2019

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

Metriche repository

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

Descrizione

Environment

  • Package version(s): Blueprint table 3.8.1
  • Browser and OS versions: Chrome

If possible, link to a minimal repro (fork this code sandbox): https://codesandbox.io/s/blueprint-sandbox-jt7py

Steps to reproduce

  1. Create blueprint table with enough columns so there is a horizontal scrollbar
  2. Move the selection to the last column using the keyboard (shift or arrow keys)
  3. Move the selection back to the first column also using the keyboard

Actual behavior

The scrollbar does not go back all the way - as you can see in the linked repo for example, when you scroll back to the start, the 'A' of the column name is cut off and you can't scroll further without using the mouse.

Expected behavior

Scrollbar should return to original position without having to use the mouse - this is an accessibility issue.

Guida contributor

Direzione di ricerca
Indaga su come il componente tabella di Blueprint gestisce la navigazione tramite tastiera e lo scorrimento orizzontale. Concentrati sul codice che aggiorna la posizione di scorrimento quando la selezione cambia tramite tastiera. Controlla se manca o è incompleta la logica di ripristino dello scorrimento.
Tech stack
typescriptreact
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70