Skip to content

[BUGFIX] Rajouter les labels sur la nav de Pix admin (PIX-17448) #12019

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

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

Conversation

theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Apr 10, 2025

🌸 Problème

Le composant PixNavigation possède désormais 2 attributs en plus (openLabel et closeLabel) qui sont affichés uniquement pour les lecteurs d'écrans. Ils ne sont pas présent sur Pix admin.

🌳 Proposition

Rajouter les labels en les traduisants au passage.

🐝 Remarques

On rajoute également l'attributs navigationAriaLabel qui n'était pas présent.

🤧 Pour tester

  • Se rendre sur Pix admin,
  • Se connecter avec le compte superadmin@example.net,
  • Se mettre en mode mobile,
  • Si la nouvelle nav n'est pas activée, exécuter cette commande pour l'activer et raffraichir la page:
scalingo -a pix-api-review-pr12019 run npm run toggles -- --key isPixAdminNewSidebarEnabled --value true
  • Sur Pix admin se mettre en mode mobile,
  • Inspecter le bouton permettant d'ouvrir la nav,
  • Constater qu'il est écrit "Ouvrir la navigation",
  • Ouvrir la sidebar,
  • Réinspecter le bouton,
  • Constater que le texte est cette fois "Fermer la navigation".

@theotime2005 theotime2005 self-assigned this Apr 10, 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 :

@theotime2005 theotime2005 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally cross-team Toutes les équipes de dev labels Apr 10, 2025
@theotime2005 theotime2005 force-pushed the pix-17448/add-labels-on-pix-admin-nav branch from 9756c89 to dd673d1 Compare April 11, 2025 12:59
@theotime2005 theotime2005 requested a review from xav-car April 13, 2025 13:51
Copy link
Member

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

func review ok !

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 👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants