Skip to content

[FEATURE] Fermer le menu utilisateur au clic souris (PIX-22649).#16110

Merged
pix-service-auto-merge merged 1 commit into
devfrom
pix-22649-close-user-menu-on-mouse-click
May 13, 2026
Merged

[FEATURE] Fermer le menu utilisateur au clic souris (PIX-22649).#16110
pix-service-auto-merge merged 1 commit into
devfrom
pix-22649-close-user-menu-on-mouse-click

Conversation

@Jeyffrey
Copy link
Copy Markdown
Contributor

@Jeyffrey Jeyffrey commented May 5, 2026

💥 Problème

Jusqu'à présent lorsqu’on cliquait sur un lien du menu utilisateur de PixApp, le menu restait ouvert.

👩‍🚀 Proposition

  • Fermer le menu au clic souris
  • Laisser le menu ouvert si l’interaction vient du clavier

♻️ Pour tester

  • Aller sur PixApp en RA
  • Tester le comportement du menu utilisateur (tout en haut à droite) à la souris et au clavier

@Jeyffrey Jeyffrey self-assigned this May 5, 2026
@Jeyffrey Jeyffrey added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally cross-team Toutes les équipes de dev labels May 5, 2026
@pix-bot-github
Copy link
Copy Markdown

pix-bot-github commented May 5, 2026

@Jeyffrey Jeyffrey force-pushed the pix-22649-close-user-menu-on-mouse-click branch 3 times, most recently from 1bf2740 to 5701537 Compare May 5, 2026 13:18
@alexandrecoin alexandrecoin added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels May 5, 2026
@action
handleTab() {
/* `setTimeout(..., 0)` is used to wait the next browser rendering and get the new focused element */
setTimeout(() => {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: on pourrait utiliser requestIdleCallback

await click(buttonMenu);

// when
await triggerEvent(screen.getByRole('link', { name: t('navigation.main.help') }), 'click', { detail: 0 });
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: on pourrait plutot utiliser triggerKeyEvent pour simuler un appuie clavier sur enter ?

Copy link
Copy Markdown
Contributor

@xav-car xav-car left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion Voir l'ajout et difficile dans un seul commit avec le déplacement de tout le code en haut.

Si tout le monde préfère avoir le template en bas. Il doit bien exister une règle de lint pour tout déplacer automatiquement

@HEYGUL HEYGUL force-pushed the pix-22649-close-user-menu-on-mouse-click branch from 5701537 to 2efbfe8 Compare May 7, 2026 14:34
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-22649-close-user-menu-on-mouse-click branch from 2efbfe8 to 3d414fa Compare May 13, 2026 08:06
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-22649-close-user-menu-on-mouse-click branch from 3d414fa to 4c2aa60 Compare May 13, 2026 08:10
@pix-service-auto-merge pix-service-auto-merge merged commit 5949f46 into dev May 13, 2026
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-22649-close-user-menu-on-mouse-click branch May 13, 2026 08:19
pix-service pushed a commit that referenced this pull request May 13, 2026
# [5.407.0](v5.406.0...v5.407.0) (2026-05-13)

### 🚀 Amélioration

- [#16201](#16201) #MDX-43 Ajout QCU découverte 2
- [#16195](#16195) Ajout de nouvelles colonnes dans la table trainings pour la recommandation de contenus formatifs (PIX-22714).
- [#16110](#16110) Fermer le menu utilisateur au clic souris (PIX-22649).
- [#16160](#16160) Importer les traductions allemande et espagnole (PIX-22665-import-translations)
- [#16146](#16146) MDX-32 Datacenter_NOV retours edito
- [#16135](#16135) MDX-43 intégrations post relectures
- [#16200](#16200) Permet la désactivation des Parcours Combiné via un Feature Toggle (PIX-22658).
- [#16172](#16172) Suppression des colonnes 'completedAt' et 'endedAt' de la table 'certification-courses' (PIX-22328)

### 🐛 Correction

- [#16173](#16173) Corriger le flaky sur la réconciliation des apprenants lors d'un import SCO (PIX-22719)
- [#16169](#16169) ne pas échouer quand un badge n'est pas valide (PIX-17427)

### 🏗️ Tech

- [#16196](#16196) 📦 Mise à jour des bibliothèques autour d'Ember (pix-22734)
- [#16199](#16199) ♻️ Fais en sorte que la `startDate` de la version soit optionnelle
- [#16203](#16203) Améliorer les logs des requêtes vers l'API LLM (PIX-22739)
- [#16202](#16202) Désactive temporairement les tests de la recette certif
- [#16096](#16096) Diminution du nombre de tests E2E tout en gardant la couverture(PIX-22657)
- [#16184](#16184) Fusionne les tests du ErrorManager et simplifie le mapping d'erreurs
- [#16170](#16170) Modification de la table `certification_versions` (PIX-22695).
- [#16165](#16165) Refactorer les jobs d'import et de validation des apprenants (PIX-22711)

### ⬆️ Montée de version

- [#16183](#16183) Update dependency eslint-plugin-n to v18 (orga)
- [#16192](#16192) Update dependency oxfmt to ^0.48.0 (api)
- [#16205](#16205) Update dependency sinon to v22 (admin)

### ☕ Autre
@pix-service
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.407.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

cross-team Toutes les équipes de dev Func Review OK PO validated functionally the PR released 🚀 Ready to Merge Tech Review OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants