palantir/blueprint

Popover v2 always take focus even if autoFocus is disabled

Open

#4 792 ouverte le 2 juil. 2021

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)TypeScript (2 167 forks)batch import
P2Package: popover2Type: bughelp wanted

Métriques du dépôt

Stars
 (20 263 stars)
Métriques de merge PR
 (Merge moyen 43j 10h) (27 PRs mergées en 30 j)

Description

Environment

  • Package version(s): "@blueprintjs/core": "^3.46.0", "@blueprintjs/popover2": "^0.11.0",
  • Operating System: Mac OS, Big Sur 11.4
  • Browser name and version: Chrome Version 91.0.4472.114 (Build officiel) (x86_64)

Code Sandbox

https://codesandbox.io/s/restless-dust-b4ggf?file=/src/App.tsx

Context

I'm trying to create a component that behave close to Suggest but i can't rely on InputGroup. I try to build my own version but i'm blocked with popover v2 because focus is always take by the popover even if autoFocus props is marked as false.

Actual behavior

Input lose focus when popover open or close event if autoFocus is disabled.

Expected behavior

When autoFocus is disabled popover is not focus when open or close.

Guide contributeur

Direction de recherche
Inspectez le code de gestion du focus du composant Popover2, en particulier les fonctions handleOpening et handleOpened, et tracez comment la prop autoFocus est utilisée. Vérifiez s'il existe une logique qui force le focus indépendamment du paramètre autoFocus.
Stack technique
typescriptreact
Domaine
frontend
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.
2
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
GitReactTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65