Skip to content

[FEATURE] Améliorer la modale d'une campagne de type moteur de recommandation (PIX-22963)#16624

Open
er-lim wants to merge 6 commits into
devfrom
PIX-22963-improve-training-modal-design
Open

[FEATURE] Améliorer la modale d'une campagne de type moteur de recommandation (PIX-22963)#16624
er-lim wants to merge 6 commits into
devfrom
PIX-22963-improve-training-modal-design

Conversation

@er-lim

@er-lim er-lim commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

🪧 Problème

Lors de la review de la PR #16325, nous avons lister des retours à traiter.

🌈 Proposition

Les faire.

✊ Remarques

  • Une librairie @formatjs/intl-durationformat a été ajouté pour gérer le polyfill de l'API durationFormat.
  • L'affichage de la durée dans la modale a été mise à jour. On a maintenant la même chose des 2 côtés.
  • Le PixAccordions a bien été mis à jour pour avoir le nouveau design ✨

🎉 Pour tester

  • Se connecter avec le compte dave-comp@example.net
  • Passer la campagne avec le code EDUMULTIP pour arriver sur la page de résultat.
  • Vérifier qu'il n'y a pas de régressions sur les durées des CFs affichées.
  • Cliquer sur une carte de CF
  • Constater que le composant PixAccordions a bien le nouveau design et que la durée s'affiche bien 🚀

Test du polyfill

  • Utiliser lambdatest avec une version de firefox < 136 (ex: 130)
  • Refaire les tests plus haut pour vérifier que la durée s'affiche bien sur la carte et dans la modale.
  • Constater que le polyfill fonctionne bien 🎉

@er-lim er-lim self-assigned this Jun 24, 2026
@er-lim er-lim added the team-devcomp Aka la Brigade du 🪩 Kool and the gang 🪩 label Jun 24, 2026
@pix-bot-github

pix-bot-github commented Jun 24, 2026

Copy link
Copy Markdown

@matthiasferraina

Copy link
Copy Markdown
Contributor

Est ce que c'est normal de ne pas avoir la duration ici ?

Capture d’écran 2026-06-24 à 14 43 19

@er-lim

er-lim commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Est ce que c'est normal de ne pas avoir la duration ici ?

Nn, je n'ai pas fait l'update ici. Je m'en occupe :)
Merci pour ton retour !

@er-lim er-lim force-pushed the PIX-22963-improve-training-modal-design branch 2 times, most recently from 51c96d9 to 07415b3 Compare June 24, 2026 13:44
@er-lim er-lim added the 👀 Func Review Needed Need PO validation for this functionally label Jun 25, 2026
@jujumathieu

Copy link
Copy Markdown
Contributor

Testé pour toutes les config de durée (j/h/m, j/h, h/m, j, rien, etc.), ça a l'air nickel.
Pour le bouton de like/dislike, quand on a cliqué sur un avis et qu'on est en focus sur l'autre on a le même affichage. Il faudrait que l'icône soit vide et entourée, comme sur ce bout de maquette.
Capture d’écran 2026-06-25 à 09 37 28

Capture d’écran 2026-06-25 à 09 41 39

Ok pour l'accordéon même si la ligne de l'accordéon non déroulé au dessus déborde un peu au hover de l'accordéon du dessous.
Capture d’écran 2026-06-25 à 09 40 18

@AndreiaPena AndreiaPena left a comment

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.

Image

@er-lim

er-lim commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Testé pour toutes les config de durée (j/h/m, j/h, h/m, j, rien, etc.), ça a l'air nickel. Pour le bouton de like/dislike, quand on a cliqué sur un avis et qu'on est en focus sur l'autre on a le même affichage. Il faudrait que l'icône soit vide et entourée, comme sur ce bout de maquette.

@jujumathieu D'accord avec toi, il faudrait que l'état focus soit représenté différemment de l'état déjà sélectionné 👍
Comme on avait déjà surchargé le composant pour rendre cela accessible sur cette page, c'est un casse-tête pour gérer les différents états 😅

@dianeCdrPix dianeCdrPix force-pushed the PIX-22963-improve-training-modal-design branch from 4777569 to 24ecd1c Compare June 25, 2026 13:05
@dianeCdrPix

dianeCdrPix commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Pour le bouton de like/dislike, quand on a cliqué sur un avis et qu'on est en focus sur l'autre on a le même affichage

@jujumathieu , on a fait les modifs pour que le focus soit différent sur les icon-buttons, quand ils ne sont pas selected :)

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

Labels

👀 Func Review Needed Need PO validation for this functionally team-devcomp Aka la Brigade du 🪩 Kool and the gang 🪩

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants