[BUGFIX] Corriger l'affichage des icônes sur les composants (PIX-15304)

[er-lim]

💥 BREAKING_CHANGES

_Décrivez ici les breaking changes (voir la doc associée)[https://ui.pix.fr/?path=/docs/d%C3%A9veloppement-breaking-changes--docs]. Préfixer la PR par [BREAKING] dans ce cas. S'il n'y en a pas, la section peut être supprimée.

🎄 Problème

Les icônes sur le composant toast ne s’affichent pas ou il y a un effet lors du hover comme on peut le voir sur la capture :

C'est aussi le cas sur d'autres composants

🎁 Proposition

Corriger cela en utilisant la propriété color plutôt que fill

🌟 Remarques

RAS

🎅 Pour tester

Vérifier, sur la RA de Pix UI, que les icônes ont bien le bon affichage 😄

[pix-bot-github]

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr761.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr761/environment

[AndreiaPena]

dev : bouton bleu et hover qui donne un fond en laissant la croix bleu

Enregistrement.de.l.ecran.2024-11-14.a.14.54.43.mov

PR : bouton noir et hover qui donne un fond avec la croix blanche

Enregistrement.de.l.ecran.2024-11-14.a.14.55.29.mov

[AndreiaPena]

Malheureusement les boutons restent noir et ne prennent pas la color plus haut.

Proposition : créer une classe sur le pixIconButton de la PixBanner et y faire un color dessus ?

[er-lim]

C'est corrigé, j'ai réutilisé le .pix-icon-button pour ce modifier et les bannières de com Pix Orga / Pix Certif.

Je te laisse vérifier 😄

[QuentinChapelain-ui]

C'est super, petite question, les hover des deux composants toaster et banner sont différents


Dans l'exemple, dans un cas, l'icône du toaster au hover est dans une couleur saturé (bleu info-700) quand dans l'autre cas en banner le hover est plus léger (red error-100). Cela-est il une contrainte technique ? Sinon je pense que ce serait mieux d'alligner les comportements sur le fond léger plutôt que saturé pour conserver la cohérence entre composant.

[er-lim]

@QuentinChapelain-ui Pas de contrainte technique, je peux faire la modification 😄