Skip to content

[BUGFIX] Corriger des erreurs suite à la montée de version Pix UI sur Pix Certif (PIX-17429). #12009

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Apr 9, 2025

🌸 Problème

Depuis la récente montée de version de Pix UI, on a rencontré plusieurs problèmes :

  • en retirant les background des boutons PixIconButton, le bouton dans l’espace surveillant apparaît en sombre alors que le fond l’est aussi, on perd énormément en contraste.
  • en ajoutant le PixOverlay dans les modales, ça à décalé certains à qui on imposait des margin en dur.
Capture d’écran 2025-04-09 à 15 30 42 Capture d’écran 2025-04-09 à 15 43 29

🌳 Proposition

Changer la couleur en surchargeant le composant Pix UI, correction temporaire en attendant une variante sur Pix UI

🤧 Pour tester

Capture d’écran 2025-04-09 à 15 33 46 Capture d’écran 2025-04-09 à 15 49 14

@AndreiaPena AndreiaPena requested a review from Agnes-V April 9, 2025 13:32
@AndreiaPena AndreiaPena self-assigned this Apr 9, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@AndreiaPena AndreiaPena changed the title [BUGFIX] Corriger la couleur du bouton dans l'espace surveillant sur Pix Certif (PIX-17429). [BUGFIX] Corriger des erreurs suite à la montée de version Pix UI sur Pix Certif (PIX-17429). Apr 9, 2025
@Agnes-V
Copy link
Contributor

Agnes-V commented Apr 10, 2025

Tests func réalisés sur sup-center-member@example.net et sur certif-pro@example.net.

  • Modale finalisation ✅
Capture d’écran 2025-04-10 à 11 20 25 Capture d’écran 2025-04-10 à 11 21 54

@AndreiaPena AndreiaPena force-pushed the pix-17429-fix-pix-icon-button-color-on-invigilator-page branch 2 times, most recently from 797b722 to 8598ea1 Compare April 10, 2025 15:54
@Agnes-V
Copy link
Contributor

Agnes-V commented Apr 11, 2025

J'ai toujours le souci pour la modale de choix de référent mais si je me mets en plein écran, ça passe. 👍🏼
Je ne sais pas si quelque chose de plus peut-être fait pour que cette modale fonctionne comme les autres mais en l'état ça marche quand même tant que l'utilisateur utilise le plein écran. 📺

Enregistrement.de.l.ecran.2025-04-11.a.11.48.01.mov

@Agnes-V Agnes-V added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Apr 11, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-17429-fix-pix-icon-button-color-on-invigilator-page branch from 8598ea1 to d9a5b6e Compare April 11, 2025 09:53
@pix-service-auto-merge pix-service-auto-merge merged commit 95c117d into dev Apr 11, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-17429-fix-pix-icon-button-color-on-invigilator-page branch April 11, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants