[BREAKING] Rendre le bouton pour ouvrir la barre de navigation accessible (PIX-16754) #833
Conversation
|
Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr833.review.pix.fr |
theotime2005
force-pushed
the
pix-16754-add-labels-to-navbar-button
branch
from
386bec5 to
9a95cfb
Compare
| @@ -58,7 +58,7 @@ module('Integration | Component | pix-navigation', function (hooks) { | |||
| test('it hides the burger menu', async function (assert) { | |||
| // when | |||
| const screen = await render( | |||
| hbs`<PixNavigation @navigationAriaLabel='label' @menuLabel='menu' />`, | |||
| hbs`<PixNavigation @navigationAriaLabel='label' @openLabel='open' @closeLabel='close' />`, | |||
| ); | |||
| assert.notOk(screen.queryByRole('button', { name: 'menu' })); | |||
There was a problem hiding this comment.
Suggestion: on pourrait utiliser screen.getByRole("menu", {name: "open"}) pour valider qu'on a le bon label quand le menu et ouvert ou fermé
There was a problem hiding this comment.
Sauf qu'on ne teste pas lorsque la taille d'écran nécessite d'afficher le bouton je crois:(
|
question : Est ce que l'attribut |
Ça dépend, est-ce qu'on veut pouvoir décider que la barre est affichée même lorsque le bouton est présent? |
theotime2005
force-pushed
the
pix-16754-add-labels-to-navbar-button
branch
from
9a95cfb to
c87d3b3
Compare
theotime2005
force-pushed
the
pix-16754-add-labels-to-navbar-button
branch
from
c87d3b3 to
6e50fa9
Compare
💥 BREAKING_CHANGES
Dans le composant PixNavigation, l'attribut "menuLabel" est remplacé par:
🎄 Problème
Pour le composant PixNavigation, on avait un simple attribut menuLabel, qui permettait d'avoir un label sur le menu, mais pas de savoir s'il était ouvert ou fermé.
🎁 Proposition
Ajouter 2 paramètres à la place de menuLabel. Le openLabel sera le label à afficher lorsque la barre est fermée et closeLabel lorsqu'elle sera ouverte.
🌟 Remarques
On a pas mis des noms par défaut aux boutons.
🎅 Pour tester