Closed xKesvaL closed 1 year ago
Heu ce que tu proposes change l'indentation de nombreux fichiers 😱😱😱
J'utilise des tab principalement (même si parfois en fonction de qui a codé il y a d'autres types d'indentations). Du coup ça fait 30 fichiers changés et beaucoup de modif qui ne sont pas dans le thème d'écriture du code.
Je ne peux pas l'accepter en l'état. Il faudrait que dans la PR finale, il n'y ait que ce que toi tu as ajouté et que ça ne touche à rien d'autre du code.
@SebL68 Oups ! J'avais pas vu ça pardon, je m'en occupe je te mentionnerais dès que j'ai tout mis à jour
"te" 😅
Il est plus simple que je refasse une PR, car si dans l'état actuel je fais un autre commit, ça sera toujours le même problème mais dans l'autre sens (on aura toujours l'impression que tout les fichiers on changé)
Évidemment, tout le code à été écrit à l'« aveugle », étant donné que je n'ai aucun moyen de le tester de manière locale. Pour autant, la majorité étant simplement du CSS, il ne devrait pas y avoir de problème.
Ce qu'il faut checker :
theme.js
color-scheme
, toutes les pages devraient être en dark-mode, mais on ne sait jamaisglobal.css
et sont plutôt claires dans leur noms