Skip to content

Resolve "correction of MR 210"

Francois AUBEUT requested to merge 215-correction-of-mr-210 into dev

Ce qui n'allait pas ? Franchement bien des choses, erreur de parcours ou precipitation :

  • Le package-lock a été remis en version 1, certainement a cause de l'utilisation de meteor npm install @mui/styles au lieu de npm install @mui/styles, il a été precisé lors de la derniere réunion dev que meteor npm n'est pas a jour
  • @mui/styles a été installé pour je ne sais quelle raison alors que tss-react est preconisé dans la doc de MUI, @mui/styles est un package de transition lors de la migration ou doit etre installé pour d'autres raisons que l'utilisation de makeStyle.
  • Par conséquent, le makeStyle fonctionnel a été remis a l'ancienne version et bougé a l'interieur du composant, des exemples de tss-react sont present partout dans le code quand j'ai fait la migration.
  • Un gros IF bien degueu est apparu de derriere les fagots en plein milieu d'un code générisé et fonctionnel pour accepter plusieurs themes. Il n'en acceptais donc plus que deux, laboite et eole
  • Grace au point précédent, Rizomo avait perdu son logo
  • Le logo doit etre precisé dans le fichier common.js de son theme.
  • La MR #210 (closed) n'avait pas été validée de notre coté, ni testé dans le theme rizomo

Bref, cette MR 215 fait un retropedalage pour annuler toutes ces erreurs et creer l'animation du spinner avec keyframes de tss-react J'en ai profité pour corriger votre logo en version mobile qui apparaissait tout petit a cause d'un gros isEoleTheme en mode fonction qui ne prenait pas en compte si c'etait en mobile. la fonction n'etait pas utile vu que la condition est un terner, donc j'en ai fait une constance

Pour finir: Je trouve des gros IF comme ca vraiment pas propre et très difficile a maintenir. Il faut simplement modifier l'image pour lui donner une marge correspondant a ce 10px ajouté en CSS

Edited by Francois AUBEUT

Merge request reports

Loading