Améliorer la composante d'onglet

[gregpetit]

Les changements suivants sont à faire dans la composante d'onglet :

• Suppression de la version small
• Modification de la couleur de fond pour les états "Sélectionné", "Survol", et "Focus sélectionné"
• Ajout du support des icônes
• Ajout de l'orientation vertical

Les changements ont été ajoutés dans les spécifications : https://confluence.interne.montreal.ca/pages/viewpage.action?pageId=141231121

[alexisletrotteur]

@gregpetit c'est un duplicatat du issue #103 ?

[gregpetit]

@alexisletrotteur Oui. C'est celle qui devrait être dans le repo angular-ui. Je vais transférer l'issue 103 vers angular-ui

[alexisletrotteur]

@gregpetit
Disponible sur dev: https://services.dev.interne.montreal.ca/boite-outils4/?p=viewall-components-onglet

[gregpetit]

@alexisletrotteur
Plusieurs corrections à faire :

• Pour la taille médium, le padding n'est pas bon. Il devrait être de 16px left et right, et de 8px top et bottom
• Pour les onglets verticaux, le texte aligné au milieu lorsqu'il va sur plusieurs lignes, alors qu'il devrait resté aligné à gauche.

- Coté accessibilité, il faudrait juste mettre la valeur "Onglets" par défaut à l'attribut aria-label de la balise nav qui englobe les onglets.

Enfin, on a ajouté des boutons de défilement quand le contenu de la barre d'onglet dépasse, afin améliorer l'accessibilité au clavier. C'est quelque chose qui est dans les specs, mais j'aurai du l,ajouter dans la liste du issue GitHub.
Est-ce que tu peux l'ajouter avec ce billet?

[alexisletrotteur]

@gregpetit J'ai fais tous les changements visuelles et a11y. Pour l'ajout des boutons de défilement c'est pas mal plus gros comme modification. Je vais devoir regarder comment on peut faire ça de façon intelligente.

Ça serait peut-être plus safe de faire un nouveau billet pour cet ajout.

[gregpetit]

@alexisletrotteur Même si le padding top et bottom est toujours à 10px, ça fonctionne au niveau de la hauteur de chaque onglet donc tu peux laisser tel quel.
Tout est beau pour le reste. Je vais créer le billet pour les boutons de flèches.

[gregpetit]

@alexisletrotteur PTI, le billet pour les boutons de flèche : #125