palantir/blueprint

Popover's boundary prop does not accept Element objects

Ouverte

#3 570 ouverte le 27 mai 2019

Voir sur GitHubGuide contributeur
 (1 commentaire) (1 réaction) (0 personne assignée)TypeScript (2 167 forks)batch import
P3Package: coreType: bughelp wanted

Métriques du dépôt

Stars
 (20 263 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Package: @blueprintjs/core@3.15.1

Issue: The boundary field of IPopoverSharedProps (defined here) does not accept values of type Element.

Description: This field (boundary) is only used as the boundariesElement field in flip and preventOverflow objects in Popper modifiers, as seen here. Both of these fields in Popper (defined here and here) accept objects of type Boundary | Element, but the boundary field defined in Blueprint only accepts Boundary.

Possible Solution: Change the type here from PopperBoundary to PopperBoundary | Element.

Guide contributeur

Direction de recherche
Trouvez la définition de type dans packages/core/src/components/popover/popoverSharedProps.ts et changez le type de boundary de PopperBoundary à PopperBoundary | Element.
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
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75