Icon size from 16px to 24px

[JF-Cozy]

Nos icones par défaut font 16px, contrairement aux icones MUI qui sont en 24px.

Cela pose un souci de cohérence sur des composants dont la taille varie selon la taille de l'icone, comme l'IconButton https://material-ui.com/components/buttons/#icon-buttons qui fait 48px chez MUI (pour une icone de 24px) contre 40px/16px.

⚠️ Penser à supprimer le fix sur l'IconButton au moment de la résolution de cette issue => https://github.com/cozy/cozy-ui/issues/2107

⚠️ Penser à supprimer le fix sur le Buttons cozy-ui/transpiled/react/buttons dans makeOverrides au moment de la résolution de cette issue. (chercher dans le code la ligne https://github.com/cozy/cozy-ui/issues/1808)

[JF-Cozy]

@joel-costa dit : 3 solutions :

• tous les icones qu'on utilise déjà passent en 24x24 => tu dois aller corriger toutes les apps et composants sur lesquels ça va casser des trucs
• on dupliques totues nos icones, on garde celles en 16x16, on fait les nouvelles en 24x24 et on corrigera au fur et à mesure pour un jour n'avoir plus que du 24x24
• on garde les 16x16 et on corrige juste l'iconButton

[joel-costa]

Et j'ajouterais que :

• 1ere solution : C'est ta responsabilité, si tu dois te taper plus d'une semaine à tout corriger tu te démerdes :p
• 2e solution : Aucune idée de l'impact en terme de temps de chargement des apps ou d'autres trucs techniques dont je n'ai pas connaissance mais mis à part ça, c'est celle que je trouve la plus séduisante
• 3e solution : La plus rapide mais on repousse le problème à plus tard ^^ (et outre le fait que les icones en 16x16 posent problème quand on importe des composants de MUI, elles sont aussi super restrictives pour moi quand je dois designer des icones. Donc avoir des icones en 24x24 m'arrangerait vachement moi aussi)

[Crash--]

2/ Si on a toutes les icones en 16x16 et 24x24, on va doubler le poids du sprite. Ça me semble pas jouable car quasiment toutes nos app continuent d'utiliser le sprite. (même les nouvelles, cf mespapiers par exemple). C'est peut-être jouable d'imaginer une solution pour que le webpack des apps découpent le sprite pour ne faire qu'un sprite avec ce dont elles ont besoin

[JF-Cozy]

@joel-costa aujourd'hui à part la taille de l'iconButton qui n'est pas en 48x48 il n'y a pas de problème plus que ça avec cette histoire d'icone ?

[joel-costa]

@JF-Cozy : Nope, de votre côté l'iconButton est le seul truc impacté par ce problème à ma connaissance. C'est surtout chez moi que ça pose problème pour quand je dois designer des icones.

[JF-Cozy]

ok j'ouvre une issue l'iconButton : https://github.com/cozy/cozy-ui/issues/2107