Skip to content
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

[TECH] Migration de PixAdmin / Composants communs et layout (PIX-15221) #10494

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

mcampourcy
Copy link
Contributor

@mcampourcy mcampourcy commented Nov 5, 2024

🍂 Problème

Les icônes ont été migrées de FontAwesome vers MaterialUi.
Pour faciliter la maintenance des icônes, un composant PixIcon a été créé dans PixUI.

🌰 Proposition

Passer toutes les icônes du scope commun dans PixAdmin sous PixIcon.

🎃 Remarques

Les noms de icônes du menu n'existent plus avec le nouveau composant, leur équivalent a été mis, avec accord des UX

🪵 Pour tester

  • Aller sur PixAdmin
  • Vérifier que les icônes du menu à gauche s'affichent correctement
  • Aller dans les Parcours Autonomes
  • Cliquer sur un parcours autonome
  • Vérifier que le fil d'Ariane s'affiche correctement
  • Aller dans les Profils Cibles
  • Choisir l'id 6001 > Clés de lecture, scroll tout en bas pour les paliers
  • Vérifier que les coches s'affichent correctement
  • Aller dans les Profils Cibles
  • Choisir l'id 1000002 > Clés de lecture, scroll tout en bas pour les paliers
  • Vérifier que les croix s'affichent correctement
  • Aller dans Profils Cibles
  • Choisir l'id 56 > Details, ouvrir la compétence 3. puis 3.1, vérifier que les icônes téléphone et tablette sont OK (vertes non barrées ou grises barrées)
  • Aller dans Profils Cibles
  • Créer un nouveau profil
  • Choisir la compétence 1. > 1.2, vérifier que les icônes téléphone et tablette sont OK (vertes non barrées ou grises barrées)

Cf le tableau ci-dessous

| | | |

Lien vers la page Avant Après
Menu gauche Capture d’écran 2024-11-07 à 09 28 25 Capture d’écran 2024-11-07 à 10 01 23
Breadcrumb Capture d’écran 2024-11-05 à 16 47 46 Capture d’écran 2024-11-05 à 16 52 18
TickOrCross Capture d’écran 2024-11-19 à 14 49 28 Capture d’écran 2024-11-19 à 14 48 18
TickOrCross Capture d’écran 2024-11-19 à 14 49 06 Capture d’écran 2024-11-19 à 14 48 45
responsive Capture d’écran 2024-11-19 à 15 29 00 Capture d’écran 2024-11-19 à 15 27 46

@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 :

@mcampourcy mcampourcy force-pushed the pix-add-pixIcon-for-admin-common branch from 6159ca4 to d3ae756 Compare November 7, 2024 08:01
@mcampourcy mcampourcy closed this Nov 7, 2024
@mcampourcy mcampourcy reopened this Nov 7, 2024
@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 :

@mcampourcy mcampourcy changed the title ♻️ admin: Add PixIcon in Breadcrumb component [TECH] admin: Add PixIcon in Breadcrumb component (PIX-15221) Nov 7, 2024
@mcampourcy mcampourcy changed the title [TECH] admin: Add PixIcon in Breadcrumb component (PIX-15221) [TECH] Migration de PixAdmin / Composants communs et layout (PIX-15221) Nov 7, 2024
@mcampourcy mcampourcy force-pushed the pix-add-pixIcon-for-admin-common branch from d3ae756 to 596ff90 Compare November 7, 2024 09:02
@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 :

@mcampourcy mcampourcy force-pushed the pix-add-pixIcon-for-admin-common branch from 596ff90 to a210551 Compare November 19, 2024 13:15
@mcampourcy mcampourcy force-pushed the pix-add-pixIcon-for-admin-common branch from a210551 to 8bb50e9 Compare November 19, 2024 13:15
@mcampourcy mcampourcy added the cross-team Toutes les équipes de dev label Nov 19, 2024
@mcampourcy mcampourcy marked this pull request as ready for review November 19, 2024 14:38
Copy link
Contributor

@lionelB lionelB left a comment

Choose a reason for hiding this comment

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

tech et func ok ✅

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.

3 participants