palantir/blueprint

Blueprint v5 popover auto placement is different from v4

Open

#5 760 ouverte le 18 nov. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)TypeScript (2 167 forks)batch import
Domain: APIPackage: popover2help 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/popover2 v1.10.0

Code Sandbox

Link to a minimal repro: https://sdlqfs.csb.app/

Problem

When configuring Popover2 with rootBoundary="viewport" or rootBoundary="document", the automatic popper.js placement has different behavior compared to Popover. This may be expected and a bugfix in some cases since the positioning engine has improved, but it may be unexpected in other cases.

There may be something we can improve here by setting / unsetting boundary and rootBoundary options on the various modifiers based on user-specified props. For example, I'm not sure that it's "correct" that we set boundary: "clippingParents" (applying the default) when the rootBoundary prop is defined: https://github.com/palantir/blueprint/blob/2460933e7084632f9f8b4f193fcda9fb9b3fd38f/packages/popover2/src/popover2.tsx#L493

Guide contributeur

Direction de recherche
Le problème décrit une différence de comportement de placement automatique entre Blueprint Popover v4 et Popover2 v5 lors de l'utilisation de la propriété `rootBoundary`. Le rapporteur pointe vers une ligne spécifique (L493) dans popover2.tsx où `boundary: 'clippingParents'` est défini même lorsque la propriété `rootBoundary` est fournie. La recherche doit examiner le code source de Popover2, en particulier la configuration des modificateurs, pour comprendre comment les options boundary et rootBoundary interagissent. L'objectif est de déterminer si le comportement actuel est intentionnel ou si des ajustements sont nécessaires pour s'aligner sur le comportement de v4. Aucune pull request ou assigné n'est présent, donc ce problème est disponible pour exploration.
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.
Une demi journée
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.
60