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

[BUGFIX] Mettre à jour et rajouter les attributs nécessaire sur la navbar de Pix app (PIX-16773) #11630

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

Conversation

theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Mar 11, 2025

🥞 Problème

On a mis à jour pix-ui pour pouvoir nommer le bouton pour ouvrir la navbar sur la page d'accueil de Pix app. On doit maintenant mettre à jour le composant et l'adapter sur Pix app.

🥓 Proposition

  • Ajouter à l'appel du composant les attributs openLabel et closeLabel en leur donnant des labels traduits,
  • Traduire ses nouveaux labels.

🧃 Remarques

Le aria-label de la navbar était écrit en dur dans le code, on l'a placé dans les traductions et traduit au passage.

😋 Pour tester

  • Se rendre sur Pix app,
  • Se connecter avec un compte comme [email protected],
  • Mettre la page en petit format (mobile),
  • En inspectant la page, constater que sur le bouton pour ouvrir la navbar est écrit "Ouvrir la navigation",
  • Cliquer dessus,
  • Constater que sur le bouton pour fermer la navbar est écrit "Fermer la navigation".

@theotime2005 theotime2005 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-evaluation PR relatives à l'expérience d'évaluation 👀 Design Review Needed cross-team Toutes les équipes de dev dependencies Pull requests that update a dependency file labels Mar 11, 2025
@theotime2005 theotime2005 self-assigned this Mar 11, 2025
@theotime2005 theotime2005 force-pushed the pix-16773-update-and-fix-navbar-for-pix-app branch from 298b365 to e4555cf Compare March 11, 2025 14:48
@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 :

@theotime2005 theotime2005 force-pushed the pix-16773-update-and-fix-navbar-for-pix-app branch from e4555cf to 918da80 Compare March 11, 2025 16:01
@theotime2005 theotime2005 force-pushed the pix-16773-update-and-fix-navbar-for-pix-app branch from 918da80 to 440beee Compare March 12, 2025 08:32
@Faraopix Faraopix added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Mar 12, 2025
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 dependencies Pull requests that update a dependency file 👀 Design Review Needed 👀 Tech Review Needed Func Review OK PO validated functionally the PR 🆘 Help needed team-evaluation PR relatives à l'expérience d'évaluation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants