Skip to content

[TECH] Utiliser FloatingUI au lieu de ember-popperjs - #1056

Merged
bpetetot merged 5 commits into
devfrom
refactor/change-popperjs-by-floating-ui
Sep 4, 2026
Merged

[TECH] Utiliser FloatingUI au lieu de ember-popperjs#1056
bpetetot merged 5 commits into
devfrom
refactor/change-popperjs-by-floating-ui

Conversation

@bpetetot

@bpetetot bpetetot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

🎄 Problème

La librairie ember-popperjs est dépréciée.

🎁 Proposition

Utiliser directement floating-ui/dom (doc) par l'intermédiaire d'un composant PixFloating copié à partir de l'implémentation de disponible dans https://github.com/universal-ember/ember-primitives.

Modifier les composants dépendants de ember-popperjs:

  • PixSelect
  • PixMultiSelect
  • PixStructureSwitcher

Supprimer la librairie ember-popperjs

🌟 Remarques

Dans une autre PR, PixTooltip pourra être réécrit en utilisant PixFloating afin de corriger les comportements buggués de la tooltip actuelle. Mais nécessitera des breaking change.

🎅 Pour tester

Tester :

  • PixSelect
  • PixMultiSelect
  • PixStructureSwitcher

Implementé sur PixApp

@bpetetot bpetetot self-assigned this Aug 28, 2026
@pix-bot-github

pix-bot-github commented Aug 28, 2026

Copy link
Copy Markdown

Choisir les applications à déployer :

@bpetetot
bpetetot force-pushed the refactor/change-popperjs-by-floating-ui branch 2 times, most recently from 9ba7469 to 051d900 Compare August 31, 2026 06:46
@xav-car

xav-car commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

ça me semble OK sur PixApp .

@bpetetot
bpetetot force-pushed the refactor/change-popperjs-by-floating-ui branch from 051d900 to b55e105 Compare September 4, 2026 08:08
@bpetetot
bpetetot merged commit 21f800a into dev Sep 4, 2026
5 checks passed
@bpetetot
bpetetot deleted the refactor/change-popperjs-by-floating-ui branch September 4, 2026 08:45
pix-service-auto-merge pushed a commit that referenced this pull request Sep 4, 2026
## [68.1.2](v68.1.1...v68.1.2) (2026-09-04)

### 🏗️ Tech

- [#1056](#1056) Utiliser FloatingUI au lieu de ember-popperjs
@pix-service-auto-merge

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 68.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants