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

[FEATURE] Ajouter la barre de navigation Modulix (PIX-14864) #10489

Merged
merged 3 commits into from
Nov 7, 2024

Conversation

clemlatz
Copy link
Member

@clemlatz clemlatz commented Nov 5, 2024

🍂 Problème

Les utilisateur·ices des modules ont des difficultés à se rendre compte où ils en sont dans la progression d'un module.

🌰 Proposition

Ajouter dans une barre de navigation flottante l'étape en cours et le nombre d'étapes totales.

🎃 Remarques

  • La barre de progression qui existe dans la maquette sera ajoutée plus tard car elle nécessite d'ajouter une option au composant Pix UI pour permettre de masquer le pourcentage.
  • Le couleur de fond de la capsule n'existe pas dans les design tokens, j'ai pris ce qui me semblait le plus proche (pix-primary-100)

🪵 Pour tester

  1. Se rendre sur le didacticiel modulix
  2. Constater le bon affichage de la barre de navigation à l'étape 1/10
  3. Passer les grains et constater la mise à jour de l'étape courante

@clemlatz clemlatz self-assigned this Nov 5, 2024
@clemlatz clemlatz changed the title [FEATURE] Ajouter la barre de navigation Modulix [FEATURE] Ajouter la barre de navigation Modulix (PIX-14863) Nov 5, 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 :

@clemlatz clemlatz force-pushed the pix-14684-add-modulix-navbar-steps branch from dde24e5 to 73b837d Compare November 5, 2024 15:50
@clemlatz clemlatz marked this pull request as ready for review November 5, 2024 15:52
@clemlatz clemlatz requested a review from a team as a code owner November 5, 2024 15:52
@clemlatz clemlatz force-pushed the pix-14684-add-modulix-navbar-steps branch from 73b837d to 1709e58 Compare November 5, 2024 15:52
@yannbertrand yannbertrand changed the title [FEATURE] Ajouter la barre de navigation Modulix (PIX-14863) [FEATURE] Ajouter la barre de navigation Modulix (PIX-14864) Nov 6, 2024
@yannbertrand
Copy link
Member

yannbertrand commented Nov 6, 2024

Quelques retours :

  • Contenu de la navbar largeur max de 740px
  • Couleur de fond à passer en --pix-primary-100 (/!\ vérifier l'accessibilité)
  • Tester avec un position: sticky pour voir si ça gère mieux le soucis de la bannière "tests internes"

@dianeCdrPix dianeCdrPix force-pushed the pix-14684-add-modulix-navbar-steps branch from 8c872ad to a0c7338 Compare November 6, 2024 13:31
@dianeCdrPix dianeCdrPix force-pushed the pix-14684-add-modulix-navbar-steps branch from 643a192 to 07edbe3 Compare November 7, 2024 10:50
@yannbertrand
Copy link
Member

LGTM 💄

dianeCdrPix and others added 3 commits November 7, 2024 14:54
Co-authored-by: Yann Bertrand <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]>
 Co-authored-by: Yann Bertrand <[email protected]>
 Co-authored-by: Clément Latzarus <[email protected]>
Co-authored-by: Yann Bertrand <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]>
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14684-add-modulix-navbar-steps branch from 07edbe3 to d467461 Compare November 7, 2024 14:54
@pix-service-auto-merge pix-service-auto-merge merged commit 4320233 into dev Nov 7, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14684-add-modulix-navbar-steps branch November 7, 2024 15: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.

5 participants