Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(global css + class lodex internes) #2038

Merged
merged 6 commits into from
Jun 28, 2024

Conversation

HeleneCreusot
Copy link
Collaborator

@HeleneCreusot HeleneCreusot commented Jun 14, 2024

  • add new htlm doc file : colors for istex design
  • add img promo
  • add img home.bg
  • add icon right-arrow
  • add new css file : tools and lodex.istex compatibility (import in istex.css : toogle line comment)
  • refactor and chore istex.css
  • add new css file : internal classes
  • add modal markdown in istex.css
  • add readme.md doc class internes theme istex

@HeleneCreusot HeleneCreusot self-assigned this Jun 14, 2024
@HeleneCreusot HeleneCreusot added the 👷‍♂️ Ready For Review PR en attente de relecture et de validation label Jun 14, 2024
add documentation use theme istex internal classes lodex
@touv
Copy link
Contributor

touv commented Jun 18, 2024

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

@HeleneCreusot
Copy link
Collaborator Author

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
Copy link
Contributor

touv commented Jun 20, 2024

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 ?

le theme istex pour data et lodex vient dans une prochaine branche
@HeleneCreusot
Copy link
Collaborator Author

@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
Copy link
Contributor

touv commented Jun 27, 2024

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

@HeleneCreusot
Copy link
Collaborator Author

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

@touv touv merged commit e4e4caf into master Jun 28, 2024
9 checks passed
@touv touv deleted the theme-istex-refactoring&class-internes branch June 28, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👷‍♂️ Ready For Review PR en attente de relecture et de validation
Projects
None yet
2 participants