Inist-CNRS / lodex

Linked Open Data EXperiment
https://www.lodex.fr/
Other
51 stars 11 forks source link

refactor(global css + class lodex internes) #2038

Closed HeleneCreusot closed 5 days ago

HeleneCreusot commented 2 weeks ago
touv commented 2 weeks ago

Ces changements vont-ils casser les instances actuelles qui utilisent le thème ISTEX et les anciens noms ?

HeleneCreusot commented 2 weeks ago

Ces changements vont-ils casser les instances actuelles qui utilisent le thème ISTEX et les anciens noms ?

@touv oui

Sauf que en début de css istex.css, on trouve : "/tools/ /@import 'istex-compatible.css';/ @import 'istex-internalclass.css'; " il faut inverser les commentaires pour faire l'import de istex-compatible.css

De plus, Je me propose pour faire la mise à jour des instances

touv commented 1 week ago

Ces changements vont-ils casser les instances actuelles qui utilisent le thème ISTEX et les anciens noms ?

@touv oui

Je comprends que ce changement soit nécessaire, mais attention à ne pas le répéter. Actuellement, nous avons 3 instances en production qui utilisent ce thème, ces instances devront être retravaillées lors de la migration, elles ne pourront donc pas être mise à jour de manière automatiquement, il faudra retravaillé le modèle. Ce qui demande du temps et de la disponibilité.

Si un nouveau changement de ce type doit se faire, il faudra créer un nouveau thème à coté de celui-ci.

Est-ce que des alias, ou double nommage ne pourraient pas être utilisés ?

HeleneCreusot commented 1 week ago

@touv : ok, le plus simple pour moi est de faire deux thèmes. Je push le theme istex sans la compatibilité. Je fais une branche pour le theme istex "data&lodex". Nous seront plus tranquilles pour la mise à jour de leur modèle.

touv commented 6 days ago

@HeleneCreusot je ne peux "merger" cette PR tant qu'il y a un conflit

HeleneCreusot commented 6 days ago

@touv : j'ai résolu les conflits directement, c'etait facile en fait