Microsoft/vscode

Cannot use arrow keys to go to toggle icon in notebook find widget

Open

#207 765 ouverte le 14 mars 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)TypeScript (10 221 forks)batch import
accessibilitybughelp wantednotebook-find

Métriques du dépôt

Stars
 (74 848 stars)
Métriques de merge PR
 (Merge moyen 11h 43m) (1 000 PRs mergées en 30 j)

Description

  1. Open a notebook and CTRL+F
  2. Use tab on the find action buttons- notice that this allows you to navigate to all of the different buttons.
  3. Use right/left arrow on the find action buttons- notice that this only loops between the first three.

I think the problem is here: https://github.com/microsoft/vscode/blob/a984153d6a98fba04f0a39a5ab9918c7b6e47511/src/vs/base/browser/ui/findinput/findInput.ts#L170-L194

The arrow key navigation is fixed at the findInput level, and we aren't adding to it, even when adding the funnel icon.

In this screencap, I accidentally press the down key-- that's not part of the demo 😅 https://github.com/microsoft/vscode/assets/31675041/9949cd63-a57c-42e3-9fd8-0d03de6f44cb

Guide contributeur

Direction de recherche
Examinez la logique de navigation par touches fléchées dans src/vs/base/browser/ui/findinput/findInput.ts autour des lignes 170-194. Le problème signale que l'icône de bascule (entonnoir) n'est pas incluse dans le cycle de navigation par touches fléchées. La correction impliquera probablement d'ajouter l'élément icône de bascule au tableau des éléments pouvant recevoir le focus dans le gestionnaire de touches fléchées.
Stack technique
typescript
Domaine
frontendaccessibility
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65